Review Request: Extend khtml API to allow retrieval of the adblock filter that was matched (for display in GUI)

Maksim Orlovich mo85 at cornell.edu
Fri Aug 21 17:09:56 BST 2009


>
> This change should be BC according to the rules at
> http://techbase.kde.org/Policies/Binary_Compatibility_Issues_With_C++
> because it only adds new non-virtual functions (but
> StringsMatcher::isMatched() becomes overloaded, so not SC).  When possible
> (KDE5?), the two overloads of this can be merged into one with a default
> argument.

StringsMatcher is a private class, so you can change methods freely, and
hence can merge the two methods right now.

I am a bit confused by the behavior of adFilteredBy when the URL is
whitelisted, though.

The KHTMLSettings new method needs @since...

Everything else looks good,
Maks






More information about the kde-core-devel mailing list