<br><br><div class="gmail_quote">2011/5/3 Laurent Aimar <span dir="ltr"><<a href="mailto:fenrir@elivagar.org">fenrir@elivagar.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On Mon, May 02, 2011 at 07:43:59PM +0300, Kaarlo Räihä wrote:<br>
> In year 2008 I sent patch that changed Windows screensaver disable behavior.<br>
> That patch was horrible, and after few years I tried making it better.<br>
><br>
> The issues it should solve are same (there are limited Windows accounts which<br>
> cannot change SPI_GETLOWPOWERTIMEOUT settings and if VLC crashes the original<br>
> settings aren't returned) but now there isn't code duplication.<br>
><br>
> It isn't perfect code wise, since I have NO clue how to create thread without<br>
> parameters using vlc thread functions. Something like<br>
><br>
> create_thread (function_to_run, id)<br>
> and<br>
> kill_thread(id)<br>
><br>
> would be perfect for this, since I don't pass any parameters. Also I ain't sure<br>
> if cancel works with vlc_thread_create, so it might work incorrectly.<br>
<br>
</div></div>There is a patch at<br>
 <a href="https://trac.videolan.org/vlc/ticket/4275" target="_blank">https://trac.videolan.org/vlc/ticket/4275</a><br>
that seems to address the screensaver issues without using thread etc<br></blockquote><div><br></div><div>AFAIK it still causes issues if VLC crashes. And some limited setups cannot change those values. Plus threading way allows additional code changes, which can be used to enable screensaver when VLC is paused (people have requested this). </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Might be worth it to have a look at it.<br>
<br>
I haven't applied it yet, as I do not know enough about the win32 API, and my questions<br>
were not answered.<br>
<br>
Regards,<br>
<font color="#888888"><br>
--<br>
fenrir<br>
</font><div><div></div><div class="h5"><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>