This is the mail archive of the elfutils-devel@sourceware.org mailing list for the elfutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] nm.c: Remove unused if/else

From: Mark Wielaard <mjw at redhat dot com>
To: elfutils-devel at lists dot fedorahosted dot org
Date: Thu, 12 May 2011 05:00:05 -0400
Subject: Re: [PATCH] nm.c: Remove unused if/else

> On 05/11/2011 10:52 AM, Mark Wielaard wrote:
> > It seems safe to just remove it.
> 
> All right, in this case we also need to remove the redundant
> parameters of `show_symbols_sysv'. Tested on x86_64-redhat-linux,
> ok to master?

Looks fine to me.

BTW. How did you test? Could that test be made part of the testsuite?

Thanks,

Mark

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]