Skip to content
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

Add support of "c++20-pmr" c++ language flavor #349

Merged
merged 14 commits into from
Oct 17, 2024

Conversation

serges147
Copy link

@serges147 serges147 commented Oct 14, 2024

  • Needed for libcyphal project where I'm planning to use it like this:
    if (CMAKE_CXX_STANDARD STREQUAL "14")
        list(APPEND LOCAL_NNVG_CMD_ARGS "--language-standard=cetl++14-17")
    else ()
        list(APPEND LOCAL_NNVG_CMD_ARGS "--language-standard=c++${CMAKE_CXX_STANDARD}-pmr")
    endif ()
  • cetl++14-17 now uses cetl::pf17::variant for union types.
  • "pmr" language feature now propagates internal allocator_ field.
  • Generation of a type members initialization a bit changed (stylistically) for easier generation (: now on a new line)

BTW, these changes are successfully in use at libcyphal PR: OpenCyphal-Garage/libcyphal#389
where I bring up registry list/access servers at Example_2_Application_0_NodeHeartbeatGetInfo_Udp .

@serges147 serges147 self-assigned this Oct 14, 2024
Copy link

sonarcloud bot commented Oct 15, 2024

@thirtytwobits thirtytwobits merged commit fbf64f0 into 3.0.preview Oct 17, 2024
236 checks passed
@thirtytwobits thirtytwobits deleted the sshirokov/3.0_preview_cetl_variant branch October 17, 2024 15:06
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.

2 participants