D9325: Use QUrl in the ScreenMapper API

Laurent Montel noreply at phabricator.kde.org
Mon Dec 18 10:33:20 UTC 2017


mlaurent added inline comments.

INLINE COMMENTS

> screenmapper.cpp:101
>      const auto screenPathWithScheme = screenUrl.url();
> -    const bool isEmpty = (path.isEmpty() || screenUrl.path() == "/");
> +    const bool isEmpty = (screenUrl.isEmpty() || screenUrl.path() == "/");
>      // restore the stored locations

"== QLatin1Char('/') ?"

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

To: amantia, #plasma, mwolff, dakon, broulik
Cc: mlaurent, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171218/e91ddede/attachment.html>


More information about the Plasma-devel mailing list