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

Issue #800: Fix clang function declaration without a prototype errors #803

Conversation

jiridanek
Copy link
Contributor

No description provided.

@jiridanek jiridanek linked an issue Oct 22, 2022 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Oct 22, 2022

Codecov Report

Base: 27.48% // Head: 27.47% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (4337714) compared to base (fd4d6af).
Patch coverage: 62.96% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #803      +/-   ##
==========================================
- Coverage   27.48%   27.47%   -0.02%     
==========================================
  Files         132      133       +1     
  Lines       31647    31653       +6     
  Branches     5063     5063              
==========================================
- Hits         8698     8696       -2     
- Misses      21847    21853       +6     
- Partials     1102     1104       +2     
Flag Coverage Δ
unittests 27.47% <62.96%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
include/qpid/dispatch/delivery_state.h 0.00% <ø> (ø)
src/adaptors/adaptor_buffer.c 0.00% <0.00%> (ø)
src/adaptors/adaptor_buffer.h 0.00% <ø> (ø)
src/adaptors/http1/http1_codec.c 0.00% <0.00%> (ø)
src/adaptors/tcp/tcp_adaptor.c 2.05% <ø> (ø)
src/delivery_state.c 0.00% <0.00%> (ø)
src/log.c 65.50% <0.00%> (-0.63%) ⬇️
src/policy.c 25.89% <0.00%> (ø)
src/router_core/delivery.h 0.00% <0.00%> (ø)
tests/clogger.c 0.00% <0.00%> (ø)
... and 23 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jiridanek jiridanek force-pushed the jd_2022_10_22_fix_clang_declaration_prototype branch from 4337714 to d995a73 Compare October 26, 2022 15:53
@jiridanek
Copy link
Contributor Author

This now reveals the Coverity issue as compile error

../src/adaptors/http_common.c: In function ‘qd_load_http_adaptor_config’:
../src/adaptors/http_common.c:99:9: error: too many arguments to function ‘qd_error_code’
   99 |         qd_error_code(QD_ERROR_CONFIG, "Invalid value for HTTP version: %s, expected 'HTTP1' or 'HTTP2'", version_str);
      |         ^~~~~~~~~~~~~
In file included from ../src/entity.h:22,
                 from ../src/adaptors/adaptor_common.h:24,
                 from ../src/adaptors/http_common.h:22,
                 from ../src/adaptors/http_common.c:20:
../include/qpid/dispatch/error.h:88:22: note: declared here
   88 | QD_EXPORT qd_error_t qd_error_code(void);
      |                      ^~~~~~~~~~~~~

@jiridanek jiridanek merged commit 095e929 into skupperproject:main Oct 26, 2022
@jiridanek jiridanek deleted the jd_2022_10_22_fix_clang_declaration_prototype branch October 26, 2022 15:58
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.

Compilation failures with clang on Fedora 37
3 participants