D9100: KUriFilter: simplify data structures, fix memory leak

Emirald Mateli noreply at phabricator.kde.org
Sat Dec 2 14:49:59 UTC 2017


emateli added inline comments.

INLINE COMMENTS

> kurifilter.cpp:597
> +        if (filters.isEmpty() || filters.contains(plugin->objectName())) {
> +            if (plugin->filterUri(data)) {
>                  filtered = true;

Perhaps this can be merged with the if statement above.

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D9100

To: dfaure, apol, davidedmundson, arichardson, bshah
Cc: emateli, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171202/bdb8937d/attachment.html>


More information about the Kde-frameworks-devel mailing list