[MPlayer-dev-eng] PATCH: signal handling in gmplayer, immediate mode in tvi_v4l, wrong warning for tv:// MRLs

[Arpi]

Hi,

> hi.
> On Tue, 20 May 2003 10:50:59 +0200
> Arpi <arpi at thot.banki.hu> wrote:
> 
> >> MJ> 1: mp_msg() in gmplayer uses GTK message boxes, and this is REALLY a
> >> MJ> stupid idea inside a signal handler! This makes gmplayer 'almost
> >> 
> >> mp_msg() in signal handler is stupid idea even without gui =/
> 
> A> hmm, i tend to agree :)
> A> so feel free to change it... to fprintf maybe
> A> btw gui users won't know what happened to mplayer when it crashing,
> A> but who cares :)
> 
> according to "info libc" fprintf is nonreentrant and should not
> be used in signal handlers.

so then what is the allowed method of printing from sighandler?

> btw, exit_sighandler in mplayer.c is "bloated" =)
> i think it should be implemented as is in mencoder.c

no, mencoder does not have sighandler at all
it does pure 'segfault' at errors with unclean exit

> (no, i cannot do it, because it lead to modifying of main() in
> mplayer.c and i simple cannot read and understand ~2.5K lines
> long function with tons of #ifdef's =/)

:)

A'rpi / Astral & ESP-team

--
Developer of MPlayer G2, the Movie Framework for all - http://www.MPlayerHQ.hu