Re: [PATCH] Use /usr/bin/nm -D

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



----- "Bernhard Walle" <bwalle@xxxxxxx> wrote:

> * Dave Anderson [2008-08-25 16:07]:
> > 
> > I'm not sure what's worse -- not doing the translation at all
> > or showing potentially bogus entries.
> 
> *arg*, yes, I didn't check for such static functions. The functions
> in
> my test were only global, so everything worked.
> 
> However, I now modified the patch to check the size (with nm -S), and
> report "unknown" in such cases. So we still have the global symbols
> right, but no "false positives".
> 
> Is that okay?

Looks OK -- I'll give it run today, and if no problems arise, 
I'll queue it as is.

Thanks,
  Dave

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility

[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux