usta added inline comments.

INLINE COMMENTS

> kfileplacesview.cpp:1348
> +        searchUrl = searchUrlForType(QStringLiteral("Video"));
> +    }
> +

won't be nice to add an else statement? (for the sake of searchUrl )

REPOSITORY
  R241 KIO

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

To: renatoo, #frameworks, #dolphin, #kde_applications, dvratil, #vdg, ngraham, 
ervin, mlaurent
Cc: ervin, usta, mlaurent, dvratil, ngraham, #frameworks

Reply via email to