Skip to content

Support C++23 #elifdef and #elifndef - #280

Open
martis42 wants to merge 1 commit into
boostorg:developfrom
martis42:support_elifdef
Open

martis42 wants to merge 1 commit into
boostorg:developfrom
martis42:support_elifdef

Conversation

@martis42

Copy link
Copy Markdown
Contributor

Fixes #130

Disclaimer: This contribution was implemented by a Coding Agent.

cpp_re.inc and strict_cpp_re.inc have been regenerated with re2c version 1.0.2 built from the tarball offered at
https://github.com/skvadrik/re2c/releases/tag/1.0.2. The diff is large because the state numbers shifted.

Disclaimer: This contribution was implemented by a Coding Agent.

`cpp_re.inc` and `strict_cpp_re.inc` have been regenerated with re2c
version 1.0.2 built from the tarball offered at
https://github.com/skvadrik/re2c/releases/tag/1.0.2.
The diff is large because the state numbers shifted.
@codecov

codecov Bot commented Oct 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 25.00000% with 42 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@38465b8). Learn more about missing BASE report.

Files with missing lines Patch % Lines
include/boost/wave/util/cpp_iterator.hpp 20.83% 14 Missing and 5 partials ⚠️
include/boost/wave/grammars/cpp_grammar.hpp 12.50% 0 Missing and 14 partials ⚠️
samples/token_statistics/xlex/xlex_lexer.hpp 25.00% 0 Missing and 3 partials ⚠️
samples/cpp_tokens/slex/cpp_slex_lexer.hpp 66.66% 0 Missing and 2 partials ⚠️
samples/list_includes/lexertl/lexertl_lexer.hpp 33.33% 0 Missing and 2 partials ⚠️
include/boost/wave/language_support.hpp 0.00% 0 Missing and 1 partial ⚠️
test/testwave/testwave_app.cpp 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #280   +/-   ##
==========================================
  Coverage           ?   33.98%           
==========================================
  Files              ?       76           
  Lines              ?     7474           
  Branches           ?     4159           
==========================================
  Hits               ?     2540           
  Misses             ?     2386           
  Partials           ?     2548           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jefftrull

Copy link
Copy Markdown
Collaborator

Thanks! As a new feature, this will have to wait for the next Boost release (see the calendar) but very happy to have it.

@martis42

martis42 commented Nov 9, 2026

Copy link
Copy Markdown
Contributor Author

I am aware this won't hit master anytime soon. But can we merge it to the develop branch, assuming review is fine. Or does this PR have to stay open and unmerged for the foreseeable time?

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.

Support elifdef and elifndef directives

2 participants