diff options
| author | letouzey | 2011-10-26 13:33:49 +0000 |
|---|---|---|
| committer | letouzey | 2011-10-26 13:33:49 +0000 |
| commit | cf21be5bfd42720bd1cc8756cfcdb388cdaebd80 (patch) | |
| tree | fc2396883396496d349445981f5a18f1c7d9fec3 /kernel/environ.ml | |
| parent | 9f2d6ca25f784bd877c65e4fe20d5a3f6aee784d (diff) | |
When checking for emptiness, use Foo.is_empty instead of (=) Foo.empty
Here Foo is Univ.constraints, Univ.universes, Evd.evar_map, Evd.Metamap
Ok, all these structures are currently ocaml's maps or similar,
with a unique empty value, and (=) can be used on them in this
particular case. But using Foo.is_empty is safer : it will work
even if the underlying representation changes. Example : for
spotting non-legitimate use of (=) we might embed a type into
a record with a functional field.
git-svn-id: svn+ssh://scm.gforge.inria.fr/svn/coq/trunk@14614 85f007b7-540e-0410-9357-904b9bb8a0f7
Diffstat (limited to 'kernel/environ.ml')
| -rw-r--r-- | kernel/environ.ml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/environ.ml b/kernel/environ.ml index 7759efb671..7a41e62c47 100644 --- a/kernel/environ.ml +++ b/kernel/environ.ml @@ -197,7 +197,7 @@ let add_mind kn mib env = (* Universe constraints *) let add_constraints c env = - if c == empty_constraint then + if is_empty_constraint c then env else let s = env.env_stratification in |