[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: [atlantik-cvs] CVS: kdegames/atlantik/client monopigator.cpp,
From:       Dirk Mueller <mueller () kde ! org>
Date:       2004-01-23 14:12:28
Message-ID: 200401231512.28913.mueller () kde ! org
[Download RAW message or body]

On Friday 23 January 2004 11:24, Rob Kaper wrote:

> I'll assume this was changed compiler behavior then[1], but didn't expect
> that either because I've gotten used to the regular "gcc 3.4 fixes" log
> entries for those..

It is not. Its just that passing a KURL().url() to a function that expects 
const KURL&, will call KURL(const QString&) again on a temporary object. 
Creation of such a temporary object is both a waste of ressources and of cpu 
time. 

The other KURL() addition was to make it explicit where KURL is used, since 
its constructors are so extremely expensive. (A few days ago, it was visible 
in certain testcases in KHTML that more time was spend in KURL() constructors 
than in all the rest of KHTML together). 


Dirk
[prev in list] [next in list] [prev in thread] [next in thread]