[Geany-Devel] Code review for feature #683
dimitar.zhekov at xxxxx
Sat Mar 1 11:01:50 UTC 2014
On Sat, 1 Mar 2014 11:28:13 +0100
Steven VALSESIA <steven.valsesia at gmail.com> wrote:
> Hi everybody !
> 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.
More information about the Devel