Skip to content
Snippets Groups Projects

introduce [default] as abbreviation for [from_option id], and use it

Merged introduce [default] as abbreviation for [from_option id], and use it
ralf/default into master
1 unresolved thread
Merged Ralf Jung requested to merge ralf/default into master
1 unresolved thread

There's some more uses in Iris, so this seems worth it. Over std++ and Iris together, the new default will actually be used more often than the old one I removed in !31 (merged).

Merge request reports

Approval is optional

Merged by Ralf JungRalf Jung 7 years ago (May 29, 2018 1:56pm UTC)

Merge details

  • Changes merged into master with 4014c929.
  • Deleted the source branch.

Pipeline #9082 passed

Stage: build

Pipeline passed for 4014c929 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
theories/option.v
26 26 Instance: Params (@from_option) 3.
27 27 Arguments from_option {_ _} _ _ !_ / : assert.
28 28
29 (* The eliminator with the identity function. *)
  • Robbert Krebbers
    Robbert Krebbers @robbertkrebbers
    Owner

    Fine with this. How many uses of the old default were there in the std++ reverse dependencies?

  • Ralf Jung
    Ralf Jung @jung
    Author Owner

    AFAIK only the 8 in Iris.

  • Ralf Jung added 1 commit

    added 1 commit

    Compare with previous version

  • Ralf Jung merged

    merged

  • Ralf Jung mentioned in commit 4014c929

    mentioned in commit 4014c929

    • Please register or sign in to reply