[vlc-devel] Re: vlc: svn commit r19326 (courmisch)
Pierre d'Herbemont
pdherbemont at free.fr
Tue Mar 20 00:01:54 CET 2007
On 15 mars 07, at 18:36, Rémi Denis-Courmont wrote:
> Le mercredi 14 mars 2007 18:10, Subversion daemon a écrit :
>> r19326 | courmisch | 2007-03-14 17:10:14 +0100 (Wed, 14 Mar 2007) | 2
>> lines Changed paths:
>> M /trunk/src/vlc.c
>>
>> Code simplification
>
> That would break the MacOS "force exit" thingy. Though sending a
> signal
> to the signal handling threads so that it would then send a SIGABRT
> looks very convoluted to me. If you want to force exit, call exit
> (), or
> otherwise _exit(), or otherwise abort(), or even raise(SIGKILL).
> That's
> more straight-forward. And if the signal comes from outside, don't put
> it in the list of handled signals in the first place. Making VLC crash
> itself when the user press Ctrl+C was really wrong.
Thanks for pointing your change. I have just committed a fix that
make "force quit" works again. This time deamon mode safe termination
is preserved. Though I think we've better add someway to know that we
are effectively in deamon mode before trapping SIGTERM (add a VLC
command line option?).
Pierre.
--
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html
More information about the vlc-devel
mailing list