Skip to content

🐛 Add missing lookahead_case_insensitive_string#144

Merged
nevans merged 1 commit intomasterfrom
add-missing-lookahead_case_insensitive_string
Apr 17, 2023
Merged

🐛 Add missing lookahead_case_insensitive_string#144
nevans merged 1 commit intomasterfrom
add-missing-lookahead_case_insensitive_string

Conversation

@nevans
Copy link
Copy Markdown
Collaborator

@nevans nevans commented Apr 16, 2023

This adds lookahead_{token}! to def_token_matchers, as a lookahead version of match, returning the value without consuming the token.

lookahead_case_insensitive_string! is a fallback for the regexp in body_type_1part. This fallback should be invoked only very rarely.

Its definition was accidentally lost at some point during a rebase of #113.

This adds `lookahead_{token}!` to `def_token_matchers`, as a lookahead
version of match, returning the value without consuming the token.

`lookahead_case_insensitive_string!` is a fallback for the regexp in
`body_type_1part`.  This fallback should be invoked nnly very rarely.

Its definition was lost when rebasing.
@nevans nevans force-pushed the add-missing-lookahead_case_insensitive_string branch from 855c29c to 07bfed0 Compare April 16, 2023 22:21
@nevans nevans requested a review from shugo April 17, 2023 13:38
@nevans nevans merged commit dfb5174 into master Apr 17, 2023
@nevans nevans deleted the add-missing-lookahead_case_insensitive_string branch April 17, 2023 13:38
@nevans nevans changed the title 🐛 Add missing lookahead_case_insensitive_string! 🐛 Add missing lookahead_case_insensitive_string Apr 17, 2023
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.

1 participant