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

[Reimar Döffinger]

Hi,
> > >>>I would recommend all developers to test it as I intend to make it
> > >>>default for everyone with a .developer file ;-)
> > >>
> > >>Reading the patch I see that you're not making it default for everyone
> > >>with .developer. You're making it obligatory for everyone with it. Is
> > >>this desireable?
> > >
> > >no, very undesirable. i hate gdb and (almost) never use it. if
> > >something crashes i rtfs or put a printf at the appropriate location.
> > 
> > 1) gdb can help to find out the appropriate location.
> > 2) Also it is just an extended signal handler and you can just type quit 
> > to make it continue normally.
> > 3) can be disabled by commmand-line parameter.
> > 
> > Anyway, here is a fixed version...
> 
> very annoying if i'm running full-screen on console with output to
> /dev/null, and suddenly the movie freezes but won't exit. i can't see
> to type anything and can't tell that gdb is running, and can't even
> tell after switching to another vc. very very evil! :)

Sure, no problem disabling it on the command-line or in the config file,
but making it default (for developers) is a way to make sure nobody has it disabled just
because of lazyness.
So, do you think it is ok to apply in its current form?

Greetings,
Reimar Döffinger