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
Enable -Wextra #4875
Enable -Wextra #4875
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4875 +/- ##
==========================================
- Coverage 89.50% 89.46% -0.04%
==========================================
Files 225 225
Lines 50191 50189 -2
==========================================
- Hits 44923 44904 -19
- Misses 5268 5285 +17
Continue to review full report at Codecov.
|
@@ -1466,7 +1466,7 @@ Datum | |||
tsl_compressed_data_recv(PG_FUNCTION_ARGS) | |||
{ | |||
StringInfo buf = (StringInfo) PG_GETARG_POINTER(0); | |||
CompressedDataHeader header = { { 0 } }; | |||
CompressedDataHeader header = { .vl_len_ = { 0 } }; |
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.
clang-15 with -Wextra doesn't like double nested braces, and gcc 4.8.5 doesn't like single braces, so this is how we have to compromise to avoid warnings
Also see this PR https://github.com/timescale/timescaledb/pull/4867/files#r1004751330
Doesn't introduce new warnings on gcc 4.8.5 as of ab0e3ca |
Our code mostly has warnings about comparison with different signedness.
It's not something we normally do for the developer tools.
Our code mostly has warnings about assignment/comparison of int types of different signedness.
Disable-check: commit-count