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