Skip to content

File::findFirstOnLine(): minor correction to the docs #194

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Dec 27, 2023

Description

Found when reviewing some code using this function. The difference in wording is significant as the original wording gave the impression that the $start token would be excluded from the search, while in reality, it is included in the search.

Note: there are no tests for this function!

Suggested changelog entry

N/A

@jrfnl jrfnl force-pushed the feature/file-findfirstonline-fix-docs branch from 8137be7 to 9be358b Compare December 27, 2023 03:08
@jrfnl jrfnl enabled auto-merge December 27, 2023 03:11
@jrfnl jrfnl merged commit 7f1c02d into master Dec 27, 2023
@jrfnl jrfnl deleted the feature/file-findfirstonline-fix-docs branch December 27, 2023 03:14
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