Merge 1.85 fixes #209
Merged
Merge 1.85 fixes #209
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- convenience.hpp is removed - no_push and branch_path are removed
…oval Handle some previously deprecated, now removed, filesystem code
* Update runner OS options so all jobs execute * Add C++20 to matrix * Add gcc-9 and msvc-14.3 (via Appveyor) to matrix
This line was inadvertently removed in 1.84 as part of another bug fix. This change restores it and improves the associated unit test.
There is no guarantee (indeed, it is unlikely) that data within the scanner buffer will be null terminated, and so the constructor that accepts a pointer and a count is appropriate.
* Check for backslash characters before the start of the new data With BOOST_WAVE_BSIZE set to 40, test t_5_002 fails because a newline appears exactly at the beginning of a newly fetched buffer, and a backslash is at the end of the previous one. In that case the escaped newline was not detected. This change will also consider three unprocessed bytes of input data, if available, prior to the new data, which is enough to detect a trigraph backslash. * Improve data range check for backslash newline The existing check considered the space required for a trigraph backslash, but not for the following LF (or CRLF) before testing for them. With BOOST_WAVE_BSIZE set to 98 this caused a segfault in t_5_001.
* Update marker before and after fill() operation in cpplexer * A basic test case for trigraph "pound" (octothorpe) on fill boundary (detects the marker issue) This change resolves #202 --------- Co-authored-by: Chris Chisolm <chris_chisolm@intuit.com> Co-authored-by: Jeff Trull <edaskel@att.net>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In this release we fix two filed bugs and some other problems: a broken CI, removed filesystem API call, and a potential vulnerability.