D18813: Filter out invalid content in lists

Aleix Pol Gonzalez noreply at phabricator.kde.org
Fri Feb 8 10:44:43 GMT 2019


apol added a comment.


  I guess the patch makes sense overall.

INLINE COMMENTS

> atticaprovider.cpp:277
> +    for (const Content &content : contents) {
> +        if (content.isValid()) {
> +            if (checker.filterAccepts(content.tags())) {

Maybe it would be easier to read if we had a `if (!content.isValid()) continue; ...`.

REPOSITORY
  R304 KNewStuff

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

To: leinir, #knewstuff
Cc: apol, kde-frameworks-devel, michaelh, ZrenBot, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190208/a6c6fef3/attachment.html>


More information about the Kde-frameworks-devel mailing list