[MPlayer-dev-eng] [PATCH] XSSaver bug & resolving patch

[Alexander Strasser]

Hi,

The Wanderer wrote:
> Alexander Strasser wrote:
> >Paul TT wrote:
> >
> >>On Wed, 23 Nov 2005 12:45:44 +0100 Paul TT
> >><paultt at hackerjournal.it> wrote:
> >>
> >>>could the wid be rechecked only when resuming from pause???
> >>>like in the patch attached?
> >>>since vo_init returns 1 if already called, it shouldnt' be
> >>>overloading now, right i am?
> >>>maybe a check of stop-screensaver value could be added too...
> >>>
> >>>(i'm testing it :-))
> >>
> >>tested, seems to work :-)
> >
> >Wait a bit more. I am testing some a different approach currently.
> 
> Did this ever go anywhere? I don't see any futher discussion or an
> obviously-related commit...

  Yes. I commited something to prevent the crash, but I didn't come
up with a clean solution for the second underlying problem: that was
someone changing xscreensaver process while mplayer is running. And
atm moment being busy with exams I don't have time to investigate
further.

  The patch in this thread is unacceptable. Drop it.

  Alex (beastd)