Re: QCString construction
Roger Larsson <[email protected]> Sat, 10 Feb 2007 18:33:24 +0100
| Newsgroups | gmane.comp.kde.devel.optimize |
|---|---|
| Message-ID | <[email protected]> |
On Saturday 10 February 2007 01:43, David Faure wrote:
> I was debugging today the memory consumption by kmail (qt3) when sending
> signed emails with large attachments. Among other things, I noticed many
> QCString(const char*) constructions in kmail, like: QCString
> KMMessage::asString() const {
> return asDwString().c_str();
> }
> and
> messagecomposer.cpp: mEncodedBody = dwPart->AsString().c_str();
The fastest string copy is not doing it at all...
Why is code like this necessary? Why returning a low level c_str?
If it is why does it have to be in a hot path?
I have been guilty of optimizing code that is rarely used resulting in
- less readable code
- more prone to errors
- harder to optimize later
- for very little real world gain...
(I will check your patches)
/RogerL