On 20 May 2015 at 00:50, Konstantin Belousov <kostik...@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
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Reply via email to