Hello, On Tue, Apr 11, 2006 at 11:06:00PM +0300, Ismail Donmez wrote:
Salı 11 Nisan 2006 00:46 tarihinde, Uoti Urpala şunları yazmıştı:
On Mon, 2006-04-10 at 20:49 +0300, Ismail Donmez wrote:
Simpler version without XOpenDisplay , suggested by fredrik.
This also removes the runtime checks that were in the original. Doesn't this mean that if compiled with support for XScreenSaverSuspend, MPlayer would fail to disable screensaver when running under an older X server?
Here is a better(best? ;) patch which checks if the extension is supported. Check is done in vo_init hence should be better than checking everytime calling saver_on/off .
Please review/comment.
Here is an updated version (actually it is basically new code). For simplicity I moved the check so it is done on every saver_on/off, it should not matter performance-wise. I also added a QueryExtension because the documentation says without it the effects of the functions are undefined. I advise you to not ignore this, because I intend to apply soon and then the KDE-specific hack _will_ be removed unless you have exceptionally good reasons not to ("KDE does not support disabling via XScreenSaverSuspend" does not count). Greetings, Reimar Döffinger