On 01/10/13 13:17, Loren Merritt wrote: > On Mon, 30 Sep 2013, Diego Biurrun wrote: > >> On Mon, Sep 30, 2013 at 04:38:01PM +0100, Derek Buitenhuis wrote: >>> On 9/30/2013 4:28 PM, Diego Biurrun wrote: >>>> Not sure what you are trying to achieve with the if, $(STRIP) should >>>> always be set ... >>>> >>>> If we bring back stripping, we'll need the rest of the portability >>>> parts from e0be794 as well. >>> >>> For now, I do not think this patch is necessary, no? >> >> Are you referring to 01/10 or the build system patch? >> >> If the latter, then I don't know what the side effect of keeping >> those labels around is ... > > The effect of keeping those labels around is making debugging harder. > Because those labels are meaningless, and complicate the disassembly. > Also, gdb can't tell the difference between them and function entrypoints. > > This new strip command is irrelevant to any usage of libav that would > have used the old fully stripped version, because the old one was for > non-debug use.
Thanks for clear this point, makes sense then having it. _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
