Eric Lee / smarc-fsl-linux-kernel

Commit 986ffe0384c38606b94947fad04e5deacb515408

Authored by Brian Norris 2015-02-18 10:18:36 +0800

Committed by Zhang Rui 2015-02-28 13:52:48 +0800

Exists in smarc_imx_lf-5.15.y and in 32 other branches

tools/thermal: tmon: silence 'set but not used' warnings

gcc complains about the 'cols' variable being unused. This is
unavoidable, given the ncurses getmaxyx() macro-based API, which wants
to assign to a variable directly, even when we're not going to use it.

Warning:

    gcc -O1 -Wall -Wshadow -W -Wformat -Wimplicit-function-declaration -Wimplicit-int -fstack-protector -D VERSION=\"1.0\"   -c -o tui.o tui.c
    tui.c: In function ‘show_dialogue’:
    tui.c:288:12: warning: variable ‘cols’ set but not used [-Wunused-but-set-variable]
      int rows, cols;
                ^

So, add a hack to get rid of that warning.

Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Acked-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>

Showing 1 changed file with 3 additions and 0 deletions Side-by-side Diff

tools/thermal/tmon/tui.c

tools/thermal/tmon/tui.c

Diff comments View file @ 986ffe0

...	...	@@ -293,6 +293,9 @@
293	293
294	294	getmaxyx(w, rows, cols);
295	295
	296	+ /* Silence compiler 'unused' warnings */
	297	+ (void)cols;
	298	+
296	299	werase(w);
297	300	box(w, 0, 0);
298	301	mvwprintw(w, 0, maxx/4, DIAG_TITLE);