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

List:       kde-devel
Subject:    Re: [PATCH] Allow/Deny some urls to be fetch in KHTMLPart
From:       John Tapsell <john () geola ! co ! uk>
Date:       2004-12-09 17:12:28
Message-ID: 200412091712.29032.john () geola ! co ! uk
[Download RAW message or body]


> 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 <<
[prev in list] [next in list] [prev in thread] [next in thread]