[MPlayer-dev-eng] [PATCH] automatic gdb attaching

[D Richard Felker III]

On Thu, Oct 21, 2004 at 06:44:51PM +0200, Reimar Döffinger wrote:
> Hi,
> 
> > > So, do you think it is ok to apply in its current form?
> > 
> > I'm not entirely convinced this is going to be that useful.  Not
> > everybody with .developer will start fixing bugs like crazy, will
> > they?  I might be wrong, though.
> 
> Ok, to make things clear: I won't try to force this on anyone, if you
> don't want it to be default I won't make it default.
> In which case the question is where it should be disabled for whom.
> When should it be disabled for compilation (i.e. not compiled in)?
> When should only the crash_debug be set to zero by default (i.e.
> disabled at runtime?

imo compiling it in but having it disabled by default at runtime is a
good solution.

> And just to make sure that you understand the purpose of this patch:
> There are a few crashes that aren't reproducible, so unless maybe you

these indicate faulty hardware, right?
or how are they non-reproducible? i don't follow.

> get core dumps (which I really hate), this patch would allow you to just
> quickly type bt and remember the whereabouts of the bug and maybe
> sometime later fix it (the SIGTRAP handling is just to allow debugging
> hangs as well). There are some bugs that only happen to me when I
> actually want to watch a video and certainly don't want to recompile

understand.

> MPlayer or something like that ;-). If MPlayer doesn't crash, you
> should not see anything at all of this patch.

yes but i don't want to have to worry about it. like i said my mplayer
normally has -g omitted and -fomit-frame-pointer so running gdb is
fairly useless.

rich