Maybe actually build a `peek_next` function into the scanner
Open
optimization
- Dominant language
- Rust
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Currently we implement `peek_next` by cloning the scanner and advancing the clone. This honestly should be perfectly fine, since the clone is cheap and we don't need to call the function much anyway, but it's worth checking.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating the scanner and its current peek_next implementation. Compare the clone-and-advance behavior with the proposed direct scanner function, including any existing scanner tests. Done means the scanner provides peek_next without unnecessary cloning while preserving its current behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- compilers
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100