[FFmpeg-devel] [PATCH] Source level debug

[Diego Biurrun]

On Sun, Mar 02, 2008 at 03:54:39PM -0500, Rich Felker wrote:
> On Sun, Mar 02, 2008 at 08:30:34PM +0000, M?ns Rullg?rd wrote:
> > Benjamin Larsson <banan at ludd.ltu.se> writes:
> > 
> > > Hi, adding -g3 provides more debug info for gdb and it makes graphical
> > > debuggers usable.
> > >
> > > Together with --disable-optimizations you get a quite good view over
> > > what happens in the code.
> > >
> > > --- configure	(revision 12275)
> > > +++ configure	(working copy)
> > > @@ -1795,7 +1797,14 @@
> > >  check_func XOpenDisplay -lX11           &&
> > >  check_func XShmCreateImage -lX11 -lXext
> > >  
> > > -enabled debug && add_cflags -g
> > > +#check the debug level
> > > +if enabled debug; then
> > > +    if enabled sldebug; then
> > > +        add_cflags "-g3"
> > > +    else
> > > +        add_cflags "-g"
> > > +    fi
> > > +fi
> > 
> > Why not simply always use -g3 if that is supported?  I see no need for
> > extra command line options.
> 
> Probably because most users do not want 500 megs of extra crap in
> their build directory (and the corresponding slowdown of compile due
> to disk io) for a feature they won't use.

I think Mans is talking about always using -g3 in debug mode instead of
having two different debug modes.

Diego