Performance problems in CMake support

Aleix aleixpol at gmail.com
Tue Apr 1 14:35:09 UTC 2008


I have removed (and commited) the QRegExp's from the envvar and the
definitions. I've tested it when loading kdebase and it doesn't make any
difference.

I still hope that using the cache it will be a little faster. :P

Bye,
Aleix

On Tue, Apr 1, 2008 at 4:11 PM, Andreas Pakulat <apaku at gmx.de> wrote:

> On 01.04.08 15:48:39, Aleix wrote:
> > If you grep QRegExp on the parser directory you will see that it is only
> on:
> > - CMakeAst, command parser:
> >     * To collect the cmake_minimum_required version value.
> >     * The same for cmake_policy (2.6)
> > - Condition evaluator:
> >     * For the MATCHES case (from cmake, can't remove it).
> > - Project Visitor:
> >     * To get the information from the envvar list (it could be changed,
> and
> > I will, but I don't think it is significant)
> >     * On the STRING command on regex-related options.
> >     * ADD_DEFINITION command: to extract info from the definitions. (it
> > could be changed, and I will, it appears quite often so maybe it could
> be
> > significant)
> >     * REMOVE_DEFINITION command: the same but barely used.
> >
> > It is used on the manager 1 time so it is not a problem as well.
> >
> > I'll remove now the few I think that could be removed but, as I said, I
> > think the big load of regex is used by the cmake scripts themselves.
>
> Ok, so maybe we should think about not using Qt's regex support, but
> instead something else, like pcre? Or check what cmake itself uses.
>
> Andreas
>
> --
> You are fighting for survival in your own sweet and gentle way.
>
> _______________________________________________
> KDevelop-devel mailing list
> KDevelop-devel at kdevelop.org
> https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20080401/7b304e3c/attachment.html>


More information about the KDevelop-devel mailing list