linux-kernel - Re: [PATCH]scripts/kconfig/nconf.gui.c Fix warning: value computed is not used

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 03 Aug 2010 07:32:35 -0700
From:	"Justin P. Mattock" <justinmattock@...il.com>
To:	Nir Tzachar <nir.tzachar@...il.com>
CC:	Michal Marek <mmarek@...e.cz>, linux-kbuild@...r.kernel.org,
	zippel@...ux-m68k.org, lkml <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH]scripts/kconfig/nconf.gui.c Fix warning: value computed
 is 	not used

On 08/03/2010 06:46 AM, Nir Tzachar wrote:
> 2010/8/3 Justin P. Mattock<justinmattock@...il.com>:
>> On 08/03/2010 02:59 AM, Michal Marek wrote:
>>>
>>> On 3.8.2010 01:16, Justin P. Mattock wrote:
>>>>
>>>> The below patch fixes some warning messages Im getting when doing
>>>> make nconfig
>>>>
>>>>   make nconfig
>>>>    HOSTCC  scripts/kconfig/nconf.gui.o
>>>> scripts/kconfig/nconf.gui.c: In function 'print_in_middle':
>>>> scripts/kconfig/nconf.gui.c:170:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c: In function 'btn_dialog':
>>>> scripts/kconfig/nconf.gui.c:300:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c:304:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c: In function 'dialog_inputbox':
>>>> scripts/kconfig/nconf.gui.c:395:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c:397:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c:399:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c:404:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c: In function 'show_scroll_win':
>>>> scripts/kconfig/nconf.gui.c:534:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c:549:2: warning: value computed is not used
>>>> scripts/kconfig/nconf.gui.c:551:2: warning: value computed is not used
>>>>    HOSTLD  scripts/kconfig/nconf
>>>> scripts/kconfig/nconf arch/x86/Kconfig
>>>>
>>>> I looked around for a way to fix these, but am not familiar with ncurses.
>>>> if there's a better way let me know and/or send a seperate fix.
>>>>
>>>> Signed-off-by: Justin P. Mattock<justinmattock@...il.com>
>>>>
>>>> ---
>>>>   scripts/kconfig/nconf.gui.c |   11 -----------
>>>>   1 files changed, 0 insertions(+), 11 deletions(-)
>>>>
>>>> diff --git a/scripts/kconfig/nconf.gui.c b/scripts/kconfig/nconf.gui.c
>>>> index a9d9344..08a5a04 100644
>>>> --- a/scripts/kconfig/nconf.gui.c
>>>> +++ b/scripts/kconfig/nconf.gui.c
>>>> @@ -167,7 +167,6 @@ void print_in_middle(WINDOW *win,
>>>>         length = strlen(string);
>>>>         temp = (width - length) / 2;
>>>>         x = startx + (int)temp;
>>>> -       wattrset(win, color);
>>>
>>> Huh? :) This change makes nconfig look black and white. I'm all for
>>> eliminating warnings, but please do not eliminate features at the same
>>> time (or submit a patch with "Subject: make nconfig black and white" and
>>> do not mask it as warning fixes).
>>>
>>> Michal
>>>
>>
>> hmm.. make nconfig is black and white even without this patch, is that
>> normal?
>
> No. It should be using colors. Which terminal are you using? Also,
> make sure you do not have the wide characters version of ncurses
> installed.

Im using xterm.. ncurses was built with --enable-widec on both machines 
which probably explains why the whole black and white look.

>
>> as for the patch not real sure how to handle these(ncurses), but at least
>> you know now.. as for masking this with warning fixes.. ouch!! that's a
>> first... these are what gcc 4.6 are giving me not the other way around my
>> friend.
>
> These warnings are all noise, but should be solved in the last patch I
> sent Michal.
>
> Cheers.
>

cool..

Justin P. Mattock
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/