-
-
Notifications
You must be signed in to change notification settings - Fork 316
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
lib: fix missing prototypes #4191
Conversation
df4bb2d
to
dfd2910
Compare
Were the changes in the PR ready on their own? I just found multiple of these when looking at the failing macOS run in #4213, that would be fixed by this PR. |
Opening this for review. Proper fix for the BLAS/LAPACK is less than trivial, and I'm looking into that. Meanwhile a compiler suppression should suffice, preventing new cases like what happened recently with
This addresses those warnings and new warnings will cause failure. |
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.
That certainly makes sense! Thanks.
* r.report: add missing prototype * lib/db: add missing prototype * CI: fail macOS runner on -Wstrict-prototypes * lib/gmath: suppress -Wstrict-prototypes for BLAS/LAPACK wrapper header
Address
-Wstrict-prototypes
warnings and make CI macOS runner fail in new cases.