diff options
author | Elliott Hughes <enh@google.com> | 2018-03-14 16:38:34 -0700 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2018-03-14 20:11:15 -0500 |
commit | dca9fcccbc192e4b33b15e4139c21c7b514f523a (patch) | |
tree | 88ba458a5da38aa45942e3d7053b676b89af47ad /toys/posix/sort.c | |
parent | 028f9bbb65c4211866c69b5e5255aecd5254b18d (diff) | |
download | toybox-dca9fcccbc192e4b33b15e4139c21c7b514f523a.tar.gz |
Fix last uninitialized warning.
clang is fine with the noreturn nature of error_exit, but only if we don't
`if (false)` it out for non-debug builds.
lib/args.c:304:18: error: variable 'temp' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
} else if (CFG_TOYBOX_FLOAT && new->type == '.') {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
external/toybox/generated/config.h:11:26: note: expanded from macro 'CFG_TOYBOX_FLOAT'
^
external/toybox/lib/args.c:308:19: note: uninitialized use occurs here
options = --temp;
^~~~
external/toybox/lib/args.c:304:14: note: remove the 'if' if its condition is always true
} else if (CFG_TOYBOX_FLOAT && new->type == '.') {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
external/toybox/lib/args.c:255:15: note: initialize the variable 'temp' to silence this warning
char *temp;
^
= NULL
Diffstat (limited to 'toys/posix/sort.c')
0 files changed, 0 insertions, 0 deletions