Skip to content

C: Inline hot-path lexer peek helpers#1319

Merged
marcoroth merged 1 commit intomainfrom
lexer-peek
Mar 7, 2026
Merged

C: Inline hot-path lexer peek helpers#1319
marcoroth merged 1 commit intomainfrom
lexer-peek

Conversation

@marcoroth
Copy link
Owner

This pull request moves frequently called lexer peek functions to static inline functions in the header. The cold-path functions remain in the lexer_peek_helpers.c file.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 7, 2026

npx https://pkg.pr.new/@herb-tools/formatter@1319
npx https://pkg.pr.new/@herb-tools/language-server@1319
npx https://pkg.pr.new/@herb-tools/linter@1319

commit: 1211c59

@github-actions
Copy link

github-actions bot commented Mar 7, 2026

🌿 Interactive Playground and Documentation Preview

A preview deployment has been built for this pull request. Try out the changes live in the interactive playground:


🌱 Grown from commit 1211c59

@marcoroth marcoroth merged commit be2b356 into main Mar 7, 2026
67 checks passed
@marcoroth marcoroth deleted the lexer-peek branch March 7, 2026 11:54
marcoroth added a commit that referenced this pull request Mar 7, 2026
Similar to #1319, this pull request inlines the hot-path `hb_string`
functions.
@marcoroth marcoroth added this to the v1.0.0 milestone Mar 8, 2026
@marcoroth marcoroth mentioned this pull request Mar 9, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant