diff options
author | Benjamin Poulain <benjamin.poulain@nokia.com> | 2010-08-18 01:28:43 +0200 |
---|---|---|
committer | Benjamin Poulain <benjamin.poulain@nokia.com> | 2010-08-18 05:04:39 +0200 |
commit | fc216c5bd5181191c867eb6b6ec45bf252c4a074 (patch) | |
tree | e1869bef163f9d2ff7bcc4cb04ac0cda02fd9239 /src/adblock/adblockrule.h | |
parent | remove superfluous space (diff) | |
download | rekonq-fc216c5bd5181191c867eb6b6ec45bf252c4a074.tar.xz |
Cleaning of the constructor of AdBlockRule
Basic cleaning and bug fixing of AdBlockRule.
The options shoud be parsed before checking if the filter is a regexp,
otherwhise a regexp with option will never match the rule.
For example:
-"/.*/$xmlhttprequest" is a valid regexp that would not be matched.
-"/.*$xmlhttprequest/" is a valid regexp without options and the option
should not have been parsed.
The matching of the option should match the last index of "$", not the
first, for the same reason as above.
Use mid() to split the vector at once instead QString::mid() +
QString::lef().
Clean the coding style to follow the conventions of KDE.
Diffstat (limited to 'src/adblock/adblockrule.h')
0 files changed, 0 insertions, 0 deletions