[MPlayer-dev-eng] Re: [PATCH] xssaver bug #338 third try

[The Wanderer]

Alexander Strasser wrote:

> Hi,
> 
> Paul TT wrote:
> 
>> On Thu, 1 Dec 2005 17:58:22 +0100 Alexander Strasser
>> <eclipse7 at gmx.net> wrote:
>> 
>>> It is not completely trivial, i'll apply a patch that will
>>> prevent MPlayer from crashing, but let xscreensaver kick in,
>>> after it was restarted.
>>> From there i will try to come up with something better.
>> 
>> my patch above does the trick :-))))
>> look at it.... .-)
> 
> I really looked at it the first time you posted it. Maybe I didn't
> make it clear though...
> 
> I really don't like the patch, it adds a big hack. Not that there are
> no other hacks in mplayer, but the idea is to get rid of them. Your
> solution even if now working tends to cause problems in future.

In line with your answer in the other thread, this patch is dropped as
well.

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

Secrecy is the beginning of tyranny.