fix `#%literal` bindings when `case` expansion applies

racket / rhombus

Rhombus programming language

Other

350 stars 62 forks source link

fix `#%literal` bindings when `case` expansion applies #450

Closed usaoc closed 10 months ago

usaoc commented 10 months ago

This uses the change in racket/racket#4856 for case expansion, which requires a very recent development version of Racket.

Fix #417.

usaoc commented 10 months ago

Opening this as a reminder, but as the commit message says, this change requires a very recent development version of Racket, so I think we should wait for a while before merging this.

mflatt commented 10 months ago

How about using meta-if-version-at-least to work (as well as before) in older Racket versions? I'm not sure how long we can or should support old versions, but maybe we should keep trying as long as it's easy enough.

usaoc commented 10 months ago

I’m okay with that. Does the current version look good?

mflatt commented 10 months ago

Yes, that looks good. Thanks!