jackfirth / jackfirth/resyntax
Suggest using `in-inclusive-range` instead of `in-range`
Nobody has claimed this yet.
- Dominant language
- Racket
- Stars
- 70
- Forks
- 11
- PR merge metrics
- No merged PRs in 30d
Description
This code:
(for ([x (in-range a (add1 b))])
body ...)
Can be refactored to this:
(for ([x (in-inclusive-range a b)])
body ...)
See this discussion with @sorawee for some tricky details:
notjack
okay soin-inclusive-rangeis harder than I thought to handle correctly, because when it's used directly in aforform the(in-inclusive-range ...)expression is never actually seen by the macro expander. the for macro rips it apart before the expander ever sees it. and resyntax searches for expressions to rewrite by observing the macro expander's behavior, so it can't see them.
sorawee
Oh, I still thought that Resyntax only analyzes surface syntax. That's not the case anymore?
This is very cool
notjack
resyntax analyzes every expression that appears during macro expansion, though it only tries to apply rules tosyntax-original?expressions - that's why resyntax can refactor things correctly even in the face of renamings and shadowings
the downside is that if an expression is invisible to the expander, then it's invisible to resyntax as well
in theory maybe it could work to reuse the disappeared uses information that macros emit for check syntax to figure out that some expressions are part of the program even though they're never expanded
sorawee
FYI: Typed Racket has a functionality to recover correspondence between surface and expanded code. Not sure if it will be helpful to you.
https://github.com/racket/typed-racket/blob/master/source-syntax/source-syntax.rkt
notjack
oh cool that's neat! that might help to go from anin-rangeidentifier in a disappeared uses property to its enclosing(in-range ...)expression
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by examining how resyntax observes macro expansion and how the for form handles in-range and in-inclusive-range expressions. Read the linked Typed Racket source-syntax.rkt for its surface-to-expanded correspondence approach. Done means the proposed refactoring can account for disappeared uses and correctly identify the enclosing in-range expression.
Written by the indexing model from the issue text.
Assessment
- Domain
- devtools, tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100