Skip to content

Rename test folder to test (for evoke), fix warnings in pedantic - #119

Open
dascandy wants to merge 1 commit into
TartanLlama:masterfrom
evokable:make-evokable-and-fix-pedantic-warnings
Open

Rename test folder to test (for evoke), fix warnings in pedantic#119
dascandy wants to merge 1 commit into
TartanLlama:masterfrom
evokable:make-evokable-and-fix-pedantic-warnings

Conversation

@dascandy

Copy link
Copy Markdown
  • remove unused parameter names
  • mark unused variables as maybe_unused
  • resolve collision with catch's STATIC_REQUIRE
  • make assignment operators return a value
  • remove the "test.cpp" file which is lacking includes and excluded from build

and rename the test folder to "test", so that evoke picks it up and runs it (rather than just building it).

Checked with @TartanLlama beforehand for OK on the changes in principle.

… remove collision with catch's STATIC_REQUIRE, make assignment operators return a value
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.

1 participant