[Geany-Devel] Code review for feature #683

Lex Trotman elextr at xxxxx
Sat Mar 1 22:35:31 UTC 2014

On 2 March 2014 01:00, Steven VALSESIA <steven.valsesia at gmail.com> wrote:
>> 1. 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.
>> 2. 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 :)

http://www.geany.org/manual/current/index.html#save-actions note the
source is rest in doc/geany.txt

>> 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 ?

Not in the UI, which is what the user sees, because it waits until the
UI is idle to do the potentially slow thing.

> Maybe I can create a gtk_thread ?

Access to the Scintilla buffer is not thread safe, so you can't simply
run the existing save code in a separate thread. If it was that simple
we would have done it ages ago :)

> See you soon :)

More information about the Devel mailing list