Review request for kio_bookmarks

Xavier Vello xavier.vello at gmail.com
Wed Sep 3 19:07:59 BST 2008


> Hi,
> It looks pretty good.  I think it can be moved to kdebase.
>
> Few comments anyway:
>
> - Why are all your method virtual?
> It's bad practice because virtual function make the virtual table bigger,
> and add a redirection when calling it, so you should try to avoid using
> virtual if possible. (even if i agree that in this case, those performance
> issues are almost negligible)
>
> - You could add yourself as author in the KAboutData
>
> - it seems to me that echoHead("bookmarks:/") generate unfinished html

All issues fixed in r856753

Regards
-- 
Xavier vello




More information about the kde-core-devel mailing list