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

A couple of compiler warnings #1056

Closed
stv0g opened this issue Feb 18, 2021 · 2 comments
Closed

A couple of compiler warnings #1056

stv0g opened this issue Feb 18, 2021 · 2 comments

Comments

@stv0g
Copy link

stv0g commented Feb 18, 2021

#19 313.9 /tmp/tmp.OCigIi/nanomsg/src/transports/inproc/sinproc.c: In function 'nn_sinproc_shutdown_events':
#19 313.9 /tmp/tmp.OCigIi/nanomsg/src/transports/inproc/sinproc.c:224:9: warning: this statement may fall through [-Wimplicit-fallthrough=]
#19 313.9   224 |         switch (type) {
#19 313.9       |         ^~~~~~
#19 313.9 /tmp/tmp.OCigIi/nanomsg/src/transports/inproc/sinproc.c:242:5: note: here
#19 313.9   242 |     case NN_SINPROC_SRC_PEER:
#19 313.9       |     ^~~~
#19 322.6 /tmp/tmp.OCigIi/nanomsg/tests/../src/utils/stopwatch.c: In function 'nn_stopwatch_init':
#19 322.6 /tmp/tmp.OCigIi/nanomsg/tests/../src/utils/stopwatch.c:55:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 322.6    55 |     int rc;
#19 322.6       |         ^~
#19 322.6 /tmp/tmp.OCigIi/nanomsg/tests/../src/utils/stopwatch.c: In function 'nn_stopwatch_term':
#19 322.6 /tmp/tmp.OCigIi/nanomsg/tests/../src/utils/stopwatch.c:65:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 322.6    65 |     int rc;
#19 322.6       |         ^~
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_init':
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:55:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.2    55 |     int rc;
#19 328.2       |         ^~
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_term':
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:65:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.2    65 |     int rc;
#19 328.2       |         ^~
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/inproc_lat.c: In function 'worker':
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/inproc_lat.c:44:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.2    44 |     int rc;
#19 328.2       |         ^~
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/inproc_lat.c: In function 'main':
#19 328.2 /tmp/tmp.OCigIi/nanomsg/perf/inproc_lat.c:66:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.2    66 |     int rc;
#19 328.2       |         ^~
#19 328.4 In file included from /tmp/tmp.OCigIi/nanomsg/perf/remote_lat.c:32:
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_init':
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:55:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.4    55 |     int rc;
#19 328.4       |         ^~
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_term':
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:65:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.4    65 |     int rc;
#19 328.4       |         ^~
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_init':
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:55:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.4    55 |     int rc;
#19 328.4       |         ^~
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_term':
#19 328.4 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:65:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.4    65 |     int rc;
#19 328.4       |         ^~
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_init':
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:55:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.5    55 |     int rc;
#19 328.5       |         ^~
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c: In function 'nn_stopwatch_term':
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/../src/utils/stopwatch.c:65:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.5    65 |     int rc;
#19 328.5       |         ^~
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/inproc_thr.c: In function 'worker':
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/inproc_thr.c:43:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.5    43 |     int rc;
#19 328.5       |         ^~
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/inproc_thr.c: In function 'main':
#19 328.5 /tmp/tmp.OCigIi/nanomsg/perf/inproc_thr.c:67:9: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
#19 328.5    67 |     int rc;
#19 328.5       |         ^~
@gdamore
Copy link
Contributor

gdamore commented Feb 19, 2021

All true.
And all below the threshold of something that I'd really invest any cycles on at this point -- nanomsg is in "sustaining" mode only -- all new work is on NNG. (Of course if you find an actual critical bug in libnanomsg, that's different. But compiler warnings like this are merely an annoyance.)

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

No branches or pull requests

2 participants