binyomen / binyomen/loxide

Maybe actually build a `peek_next` function into the scanner

Open
#15 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.