diff options
| author | Pierre Roux | 2020-04-24 21:52:37 +0200 |
|---|---|---|
| committer | Pierre Roux | 2020-05-02 20:37:50 +0200 |
| commit | 4c39126f0a0a97152f67a3a5e7c86db860f48e39 (patch) | |
| tree | 568955642bf955fb03e51f364d3561dbd681440f /doc | |
| parent | cacb6fed6dea278f46f83c1657f4a8c3c98817db (diff) | |
Fix #12159 (Numeral Notations do not play well with multiple scopes for the same inductive)
Numeral Notations now play better with multiple scopes for the same
inductive. Previously, when multiple numeral notations where defined
for the same inductive, only the last one was considered for
printing. Now, we proceed as follows
1. keep only uninterpreters that produce an output (first
List.map_filter)
2. keep only uninterpretation for scopes that either have a scope
delimiter or are open (second List.map_filter)
3. the final selection is made according to the order of open scopes,
(find_uninterpretation) or or according to the last defined
notation if no appropriate scope is open (head of list at the end)
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/changelog/03-notations/12163-fix-12159.rst | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/doc/changelog/03-notations/12163-fix-12159.rst b/doc/changelog/03-notations/12163-fix-12159.rst new file mode 100644 index 0000000000..978ed561dd --- /dev/null +++ b/doc/changelog/03-notations/12163-fix-12159.rst @@ -0,0 +1,11 @@ +- **Fixed:** + Numeral Notations now play better with multiple scopes for the same + inductive type. Previously, when multiple numeral notations were defined + for the same inductive, only the last one was considered for + printing. Now, among the notations that are usable for printing and either + have a scope delimiter or are open, the selection is made according + to the order of open scopes, or according to the last defined + notation if no appropriate scope is open + (`#12163 <https://github.com/coq/coq/pull/12163>`_, + fixes `#12159 <https://github.com/coq/coq/pull/12159>`_, + by Pierre Roux, review by Hugo Herbelin and Jason Gross). |
