apol added a comment.

  LGTM +1

INLINE COMMENTS

> Viewer.cpp:131
>          {
> -            for (int i = 0; i < args->count(); ++i)
> +            for (int i = 0; i < urlArgs.count(); ++i)
>              {

Prefer foreach to iterate a QList

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: volkov, Plasma
Cc: apol, plasma-devel, sebas
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to