[Differential] [Accepted] D2542: Simplify code

mart (Marco Martin) noreply at phabricator.kde.org
Tue Aug 23 08:25:29 UTC 2016


mart accepted this revision.
mart added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> kirigamiplugin.h:85
>      {
> -        #ifdef Q_OS_WIN
> -        // On Windows, the leading slash does not make sense, so let's snip that off
> -        return baseUrl().path().mid(1) + QLatin1Char('/') + path;
> -        #else
> -        return baseUrl().path() + QLatin1Char('/') + path;
> -        #endif
> +        return baseUrl().toLocalFile() + QLatin1Char('/') + path;
>      }

even tough tolocalfile should always behave correctly, did you actually try this on windows? a bit uneasy to remove platform specific code without testing it on that platform

REPOSITORY
  rKIRIGAMI Kirigami

BRANCH
  master

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

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

To: apol, #kirigami, mart
Cc: plasma-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160823/c236ded2/attachment.html>


More information about the Plasma-devel mailing list