Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 20 May 2015 07:56:29 -0700
From:      Adrian Chadd <adrian@freebsd.org>
To:        Konstantin Belousov <kostikbel@gmail.com>
Cc:        freebsd-current <freebsd-current@freebsd.org>
Subject:   Re: libc/gen: unused variables
Message-ID:  <CAJ-Vmo=e6C1FSbq8_26BT7%2BHc-NHQyOBZ2%2BuTysx8sHnQ33qvw@mail.gmail.com>
In-Reply-To: <20150520075030.GJ2499@kib.kiev.ua>
References:  <CAJ-Vmok3-tDX-9X4NOjghoX40BQpF12XPL9S%2B=bsY5pkYuP39w@mail.gmail.com> <20150520075030.GJ2499@kib.kiev.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
On 20 May 2015 at 00:50, Konstantin Belousov <kostikbel@gmail.com> wrote:
> On Wed, May 20, 2015 at 12:11:08AM -0700, Adrian Chadd wrote:
>> Hi,
>>
>> here's a patch against -head that eliminates gcc-4.9 complaining about
>> set-but-unused variables. It just comments them out for now - are
>> these okay to eliminate, or is there something else that's
>> broken/missing?
> Of course, the patch as posted must not be committed.

Absolutely. This patch was more like "is this all dead code, or are we
supposed to be checking these fields and we aren't?"

> For dl_init_phdr_info(), it is acceptable to eliminate AT_PHENT case
> and phent local.

Would you like to do the elimination, so I don't mess it up? :)

> I have not looked at the other places.

Thanks though!



-adrian



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-Vmo=e6C1FSbq8_26BT7%2BHc-NHQyOBZ2%2BuTysx8sHnQ33qvw>