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 1/4] default_debugscn_p: Add support for compressed debug section.


> On Thu, Aug 23, 2012 at 04:52:58PM -0700, Roland McGrath wrote:
> > This is completely orthogonal to your MIPS-related changes, isn't it?
> > I think Mark is on top of the issues about fixing this issue.  But it
> 
> The reason I did this is because readelf.c does the same thing,
> and I think I would cause a regression there if I didn't.

I see your point.  I think the upshot of Mark's analysis is that it's not
meaningfully risky to do this change.  The one caller that doesn't handle
it well is already pretty scrod when faced with any .zdebug_* sections.  So
unless Mark objects, I think this is fine (I don't think it matters much
one way or the other to put it in #if USE_ZLIB).


Thanks,
Roland

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