[vlc-devel] Re: vlc: svn commit r13443 (damienf)
Damien Fouilleul
Damien.Fouilleul at laposte.net
Tue Nov 29 10:23:40 CET 2005
Gildas,
I thought about it, and IMHO the risk are not that great since:
1) the system settings changes are not persisted and only apply to the
current user (as per API call)
2) other VLC instances will find out that the screensaver is already
disabled (timeouts set to 0) and will not attempt to change those
settings again, nor restore them. Obvioulsy, problem is when the first
app quits, screensaver settings will be restored and other playing
instances will be affected.
As far as I can tell, the only critical problem is that VLC crashes
while playing video and do not restore the settings,but since they are
not persisted a simple logout or reboot should restore original settings.
Damien
Gildas Bazin wrote:
>On Monday 28 November 2005 22:02, Subversion daemon wrote:
>
>
>>r13443 | damienf | 2005-11-28 23:02:13 +0100 (Mon, 28 Nov 2005) | 2 lines
>>Changed paths:
>> M /trunk/modules/video_output/directx/directx.c
>>
>>directx.c: disable screensaver by temporarily resetting screensaver system
>>
>>
>settings
>
>
>
>IMHO this might not be very wise.
>
>If I'm not mistaken, you are changing system wide settings there, so try to
>imagine what happens when you have several instances of VLC (or vouts)
>running at the same time.
>
>--
>Gildas
>
>
>
--
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