<div dir="ltr"><div><br>
> 1. is it necessary to remove the reference to gthread-2.0 in the<br>> 
makefile for your changes? If not then make that a separate change<br>> 
with its justification. Never hide unrelated changes inside other<br>> 
changes.<br><br></div>Ok, I'll take care of that in the future.<br><div>
<br>
> 2. Since you have added a preference setting it needs to be documented.<br><br></div><div>I'll check the doc to know how to doc the doc, because I don't know what you're talking about :)<br></div><div><br>
> The method suggested by Dimitar will still block, but not until the UI<br>
> has become idle, so hopefully it will be less apparent to users.<br><br>A plugin_idle_add(func) won't create latency ?<br></div><div>Maybe I can create a gtk_thread ?<br></div><div><br></div><div>See you soon :)<br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-03-01 12:24 GMT+01:00 Lex Trotman <span dir="ltr"><<a href="mailto:elextr@gmail.com" target="_blank">elextr@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On 1 March 2014 22:16, Steven VALSESIA <<a href="mailto:steven.valsesia@gmail.com">steven.valsesia@gmail.com</a>> wrote:<br>
>> First, the "editor-notify" signal is a tight spot, all Scintilla<br>
>> events go there. Blocking them while saving may not be a good idea,<br>
>> especially considering that a file save error may display a modal<br>
>> dialog IIRC. You can do a plugin_idle_add(func) instead, and make<br>
>> func() return FALSE.<br>
><br>
> I think I don't understand, where do I block these signals ?<br>
> My implementation of editor-notify() return FALSE, so it doesn't block<br>
> anything, no ? :)<br>
<br>
</div>What Dimitar is referring to is that saving a file is a slow<br>
operation, and it will block the UI while it happens since your<br>
handler calls the save operation synchronously.  And since this is a<br>
"save on unfocus" that means the user moved the cursor somewhere<br>
unrelated and that is now possibly being blocked.  Some people insist<br>
on editing large files over slow remote links :( and so we can't<br>
always assume that file saving is instantaneous.<br>
<br>
The method suggested by Dimitar will still block, but not until the UI<br>
has become idle, so hopefully it will be less apparent to users.<br>
<br>
Cheers<br>
<span class="HOEnZb"><font color="#888888">Lex<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
>> Second, mentioning -lgthread-2.0 twice may be required, depending on<br>
>> the libraries and linker, and won't do any harm otherwise.<br>
><br>
> Ok, roger that !<br>
><br>
><br>
> 2014-03-01 12:01 GMT+01:00 Dimitar Zhekov <<a href="mailto:dimitar.zhekov@gmail.com">dimitar.zhekov@gmail.com</a>>:<br>
><br>
>> On Sat, 1 Mar 2014 11:28:13 +0100<br>
>> Steven VALSESIA <<a href="mailto:steven.valsesia@gmail.com">steven.valsesia@gmail.com</a>> wrote:<br>
>><br>
>> > Hi everybody !<br>
>><br>
>> Hi.<br>
>><br>
>> > Please, take a look to my patch concerning the feature request #683.<br>
>><br>
>> A disclaimer first: I'm not using the autosave actions plugin.<br>
>> The code seems to match autosave closely, with no obvious errors.<br>
>><br>
>> > Let me know if you see how I can improve my code :)<br>
>><br>
>> First, the "editor-notify" signal is a tight spot, all Scintilla<br>
>> events go there. Blocking them while saving may not be a good idea,<br>
>> especially considering that a file save error may display a modal<br>
>> dialog IIRC. You can do a plugin_idle_add(func) instead, and make<br>
>> func() return FALSE.<br>
>><br>
>> Second, mentioning -lgthread-2.0 twice may be required, depending on<br>
>> the libraries and linker, and won't do any harm otherwise.<br>
>><br>
>> --<br>
>> E-gards: Jimmy<br>
>> _______________________________________________<br>
>> Devel mailing list<br>
>> <a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
>> <a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> Devel mailing list<br>
> <a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
> <a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
><br>
_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
</div></div></blockquote></div><br></div>