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>