-
Notifications
You must be signed in to change notification settings - Fork 274
Silence warnings resulting from catch.hpp [blocks: #2310, #3425] #3876
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
Conversation
82efc5c
to
472c69e
Compare
Wrap catch.hpp in a custom header use_catch.h that silences warnings, and make all files use that header file instead.
This is to avoid direct inclusion of catch.hpp when use_catch.h should be used instead. The first guard here is module_dependencies.txt, where only testing-utils must have "catch" listed. The second line of defense is a macro that must be set.
Do not impose formatting rules on imported code.
472c69e
to
606ffdf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫
This PR failed Diffblue compatibility checks (cbmc commit: 606ffdf).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/98117729
Status will be re-evaluated on next push.
Please contact @peterschrammel, @thk123, or @allredj for support.
Common spurious failures:
- the cbmc commit has disappeared in the mean time (e.g. in a force-push)
- the author is not in the list of contributors (e.g. first-time contributors).
The incompatibility may have been introduced by an earlier PR. In that case merging this
PR should be avoided unless it fixes the current incompatibility.
Would there be merit in upstreaming these changes to catch.hpp? |
This will need a manual TG bump - will create. |
As in: fixing catch.hpp not to produce any warnings? Possibly so, but through the changes propose here we at least don't take any dependency on the proposals actually being picked up. Half-related: I have a commit in the waiting to update catch.hpp to the most recent version, which will address the one local change we had made to catch.hpp earlier. |
Thanks @thk123! Please just merge this PR when the corresponding TG change is ready. |
@tautschnig done. |
This is factored out from #3425 in an attempt to get at least this bit approved and merged.