> I know this can be not obvious, the url in KHTMLPart::insertBlacklistUrl
> can be a KURL, but i don't think this is useful to store a full class. With
> a qstring or a char pointer, the list can take less memory and should be
> use directly by some widget without convertion. But i agree, this can be
> changed.

Or just add a comment documenting it :)  Just make it clear what your 
intention is.

Same with the other - it's that what is meant to happen, say that in the .h 
doxygen comment.

Btw, good to see new developers on khtml.  Keep up the good work :)

JohnFlux
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<