Rename seal lemmas from `_eq` to `_unseal` and make sealing stuff `Local`.
As discussed in iris!778 (comment 80612)
Merge request reports
Activity
I renamed them into
_seal_eq
to be consistent with the projection of theseal
record.What's a bit odd though, since what they are doing is unsealing?
seal_eq
is called that because it is about@seal _ = _
. Butnclose_seal_eq
hides the@seal
behind aDefinition
so I don't think this still makes sense.FWIW just
nclose_unseal
would be a good name IMO, without_eq
-- stating the fact that this is an equality seems somewhat redundant.- Resolved by Robbert Krebbers
I'm fine with just
unseal
.How shall we call the tuples with unseal lemmas:
Definition unseal_eqs := (uPred_pure_eq, uPred_and_eq, uPred_or_eq, uPred_impl_eq, uPred_forall_eq, uPred_exist_eq, uPred_internal_eq_eq, uPred_sep_eq, uPred_wand_eq, uPred_plainly_eq, uPred_persistently_eq, uPred_later_eq, uPred_ownM_eq, uPred_cmra_valid_eq, @uPred_bupd_eq).
If we drop
_eq
from the lemma names,_eqs
no longer makes much sense. Maybe, since it's in a module, justunseal
is OK?
added 3 commits
-
99c9226f...b7a5fed7 - 2 commits from branch
master
- 7f028b22 - Rename seal lemmas from `_eq` to `_unseal` and make sealing stuff `Local`.
-
99c9226f...b7a5fed7 - 2 commits from branch
I now also added the
Add Search Blacklist
for_unseal
.Edited by Robbert Krebbersmentioned in merge request iris!793 (merged)
- Resolved by Robbert Krebbers
What's our policy w.r.t. a CHANGELOG here?
These are internal definitions, so nobody should depend on them. However, there is at least one reverse dependency (= Iris) that breaks the abstraction, and other's Iris forks might do that too.
- Resolved by Robbert Krebbers
added 8 commits
-
27c34097...53c9d7f7 - 5 commits from branch
master
- 85d5291f - Rename seal lemmas from `_eq` to `_unseal` and make sealing stuff `Local`.
- c215b6cc - Hide `_unseal` in results of `Search`.
- 458a3fa8 - CHANGELOG.
Toggle commit list-
27c34097...53c9d7f7 - 5 commits from branch
enabled an automatic merge when the pipeline for 458a3fa8 succeeds
mentioned in commit ebb89887