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

warning: type of 'qd_server_listener' does not match original declaration [-Wlto-type-mismatch] #343

Open
jiridanek opened this issue Apr 16, 2022 · 2 comments · May be fixed by #355
Open
Assignees
Labels
bug Something isn't working

Comments

@jiridanek
Copy link
Contributor

This does not make any sense to me

/tmp/skupper-rpms/skupper-router/src/server_private.h:54:16: warning: type of 'qd_server_listener' does not match original declaration [-Wlto-type-mismatch]
 qd_listener_t *qd_server_listener(qd_server_t *server);
                ^
/tmp/skupper-rpms/skupper-router/src/server.c:1613:16: note: 'qd_server_listener' was previously declared here
 qd_listener_t *qd_server_listener(qd_server_t *server)
                ^
/tmp/skupper-rpms/skupper-router/src/server.c:1613:16: note: code may be misoptimized unless -fno-strict-aliasing is used
/tmp/skupper-rpms/skupper-router/include/qpid/dispatch/server.h:573:6: warning: type of 'qd_listener_listen' does not match original declaration [-Wlto-type-mismatch]
 bool qd_listener_listen(qd_listener_t *l);
      ^
/tmp/skupper-rpms/skupper-router/src/server.c:1654:6: note: 'qd_listener_listen' was previously declared here
 bool qd_listener_listen(qd_listener_t *li) {
      ^
/tmp/skupper-rpms/skupper-router/src/server.c:1654:6: note: code may be misoptimized unless -fno-strict-aliasing is used
/tmp/skupper-rpms/skupper-router/src/server_private.h:58:6: warning: type of 'qd_listener_decref' does not match original declaration [-Wlto-type-mismatch]
 void qd_listener_decref(qd_listener_t* ct);
      ^
/tmp/skupper-rpms/skupper-router/src/server.c:1661:6: note: 'qd_listener_decref' was previously declared here
 void qd_listener_decref(qd_listener_t* li)
      ^
/tmp/skupper-rpms/skupper-router/src/server.c:1661:6: note: code may be misoptimized unless -fno-strict-aliasing is used
/tmp/skupper-rpms/skupper-router/src/router_core/router_core_private.h:889:6: warning: type of 'qdr_action_enqueue' does not match original declaration [-Wlto-type-mismatch]
 void qdr_action_enqueue(qdr_core_t *core, qdr_action_t *action);
      ^
/tmp/skupper-rpms/skupper-router/src/router_core/router_core.c:441:6: note: 'qdr_action_enqueue' was previously declared here
 void qdr_action_enqueue(qdr_core_t *core, qdr_action_t *action)
      ^
/tmp/skupper-rpms/skupper-router/src/router_core/router_core.c:441:6: note: code may be misoptimized unless -fno-strict-aliasing is used
gmake[2]: Leaving directory '/tmp/skupper-rpms/skupper-router/x86_64-redhat-linux-gnu'
@jiridanek
Copy link
Contributor Author

jiridanek added a commit to jiridanek/skupper-router that referenced this issue Apr 19, 2022
@kgiusti kgiusti added the bug Something isn't working label Apr 22, 2022
@kgiusti kgiusti added this to the 2.1.0 milestone Apr 22, 2022
@jiridanek jiridanek linked a pull request Apr 23, 2022 that will close this issue
@jiridanek
Copy link
Contributor Author

jiridanek commented Apr 29, 2022

This warning appears on CentOS 8, but not on Fedora 36. Only the c_unittests binary is affected.

Looks like tied to gcc version. Here is failure with gcc-8 that goes away with gcc-11, FreeRDP/FreeRDP#5358

jiridanek added a commit to jiridanek/skupper-router that referenced this issue May 2, 2022
@ganeshmurthy ganeshmurthy modified the milestones: 2.1.0, 2.2.0 Sep 30, 2022
@ganeshmurthy ganeshmurthy modified the milestones: 2.2.0, 2.3.0 Nov 11, 2022
@ganeshmurthy ganeshmurthy modified the milestones: 2.3.0, 2.4.0 Feb 6, 2023
@ganeshmurthy ganeshmurthy modified the milestones: 2.4.0, 2.5.0 Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants