KSycocaFactory question

David Faure faure at kde.org
Wed Dec 7 09:27:26 GMT 2005


On Wednesday 07 December 2005 08:02, Thomas Braxton wrote:
> It doesn't seem to be usefull to allocate offsetList just to delete it right 
> away. Why not just get rid of the wasteful allocation and deletion. There 
> also doesn't seem to be a reason to iterate through two loops when one can 
> clearly suffice. This code seems to also be copy/pasted to 
> kio/kio/kservicefactory.cpp
> Is it OK if I change these to the second form or am I missing something?

I think you're right, please go ahead.

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).





More information about the kde-core-devel mailing list