[Geany-devel] Performance issues? [PATCH]

[Colomban Wendling]

Le 27/03/2011 13:23, Yura Siamashka a écrit :
> On Sun, 27 Mar 2011 14:15:45 +1100
> Lex Trotman <elextr at gmail.com> wrote:
> 
>>> 1) geanyprj act only on "document-open", "document-save", "document-activate" callbacks
>>> 2) geanyprj add a lot of TMWorkObject objects using tm_workspace_add_object() to Geany
>>
>> And tm_workspace_add_object sets the parent to be the whole workspace!
> 
> Can something be done for this?

No idea -- I only barely know this part if the TagManager, sorry.

>> Because of item 2) above I think this will reparse all open files.
> 
> I don't think it is true. I used mantisbt project to test. Initially it takes few seconds
> to load the project (parse all files). Delay during typing is much less then second. 

You are right, because updating the root workspace don't really update
the child SourceFiles (because the force param is set to FALSE).
Projects are maybe affected though (however, not sure where/when they
are used).

> Anyway here is patch that make Geany don't parse tags when user is actively typing.

I'm not sure about this patch, because it is then really easy to make
the tag list never update automatically.
E.g. if update timeout is set to 1s, just type something every 999ms and
then update will never happen, even if there is actually 1s free to do it.

Are really the update making something unresponsive for you?

But maybe I'm worrying too much and this only need the timeout to be
configured otherwise, I may try to tune this if you really think it's
important.

I also plan to try to move the TagManager parsing in a separate thread
at some point, maybe it'll help (though maybe it's the UI update that
takes most of the time...). However, don't expect this to happen very
soon, I'm a thread newbie so it's likely to take me time to get it
working properly.

Cheers,
Colomban