src/module/e_mod_main.c:564:6: warning: variable 'w' is used uninitialized whenever switch default is taken [-Wsometimes-uninitialized]
default:
^~~~~~~
src/module/e_mod_main.c:568:36: note: uninitialized use occurs here
e_gadcon_client_aspect_set(gcc, w, h);
^
src/module/e_mod_main.c:547:9: note: initialize the variable 'w' to silence this warning
int w, h;
^
= 0
src/module/e_mod_main.c:564:6: warning: variable 'h' is used uninitialized whenever switch default is taken [-Wsometimes-uninitialized]
default:
^~~~~~~
src/module/e_mod_main.c:568:39: note: uninitialized use occurs here
e_gadcon_client_aspect_set(gcc, w, h);
^
src/module/e_mod_main.c:547:12: note: initialize the variable 'h' to silence this warning
int w, h;
^
= 0Details
Details
Compiled, no warnings.
Alternatively, we could initialize w, h to zero as Clang suggests. But I don't know that that is meaningful.
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
no - return .. no... or just move default up so it sets w and h to 16 always. :) as such we dont use the orientation cases here as we're always "square"