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

Warnings fix after c++17 #1691

Closed
wants to merge 30 commits into from
Closed

Warnings fix after c++17 #1691

wants to merge 30 commits into from

Conversation

dcoeurjo
Copy link
Member

@dcoeurjo dcoeurjo commented Mar 5, 2023

PR Description

Checklist

  • Unit-test of your feature with Catch.
  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug mode.
  • All continuous integration tests pass (Github Actions & appveyor)

@dcoeurjo dcoeurjo added the Build label Mar 5, 2023
@dcoeurjo
Copy link
Member Author

dcoeurjo commented Mar 7, 2023

Merged into #1689
closing this one

@dcoeurjo dcoeurjo closed this Mar 7, 2023
@dcoeurjo dcoeurjo deleted the warningsc++17 branch March 7, 2023 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant