From kfm-devel Mon Sep 21 14:53:09 2009 From: "Maksim Orlovich" Date: Mon, 21 Sep 2009 14:53:09 +0000 To: kfm-devel Subject: Re: Review Request: khtml adblock filter lookup API Message-Id: <61804.24.58.22.16.1253544789.squirrel () webmail ! cornell ! edu> X-MARC-Message: https://marc.info/?l=kfm-devel&m=125354748517107 > Dear all, > > I've seen it suggested on another ML that here is the place to discuss > KHTML changes/additions. If you weren't aware of it already, would it > be possible for an expert to look at http://reviewboard.kde.org/r/1382 > and give a yes/no whether it is acceptable? > > This is an addition to the KHTML API (binary and source compatible) to > allow the adblock filter matching a URL to be identified - the current > API provides only a boolean match. This is so that the matching > filter can be shown in the adblock dialogue, allowing the filter that > caused something to be blocked to be identified. There should be no > performance impact under normal operation. Looks fine, though I can't be 100% sure that what it returns in complex cases is identical to the input. Sorry for missing your previous e-mail.