Skip to content

fix: Improve Reader internal code by using with statements#171

Merged
tmathern merged 13 commits intomainfrom
mathern/with-stements-review
Sep 22, 2025
Merged

fix: Improve Reader internal code by using with statements#171
tmathern merged 13 commits intomainfrom
mathern/with-stements-review

Conversation

@tmathern
Copy link
Collaborator

  • Improve Reader internal code by using with statements in more places (perf overhead is minimal, confirmed by banchmark)

@tmathern tmathern self-assigned this Sep 22, 2025
@tmathern tmathern marked this pull request as ready for review September 22, 2025 03:33
@tmathern tmathern requested review from dyro and gpeacock September 22, 2025 03:33
Copy link
Contributor

@dyro dyro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks clean!

@tmathern tmathern merged commit 52cbec9 into main Sep 22, 2025
22 checks passed
@tmathern tmathern deleted the mathern/with-stements-review branch September 22, 2025 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants