search

melisgl / named-readtables

The official repo of named-readtables.
Other
66 stars 15 forks source link

:modern doesn't make much sense in defreadtable #12

Closed ailisp closed 4 years ago

ailisp commented 6 years ago

When defining a readtable with (:merge :modern) without (:case :preserve), it still a "upcase" readtable:

ILLUSION> (defreadtable illusion-readtable
                      (:merge :modern))
{10027E0273}>.
#<NAMED-READTABLE ILLUSION-READTABLE {10027E0273}>
ILLUSION> *readtable*
#<NAMED-READTABLE ILLUSION-READTABLE {10027E0273}>
ILLUSION> (readtable-case *readtable*)
:UPCASE
melisgl commented 4 years ago

:MERGE and MERGE-READTABLES-INTO merge macro chars only and signal an error on conflict. It's unclear what the right semantics for merging CASE would be. Updated the documentation in 08c0df90d747220414eda3e0888fefeed2e40f0f to reflect this state of affairs.

ailisp commented 4 years ago

Thanks! New document looks very clear. I was assume MODERN means a case preserve read table.