D14844: [CompressFileItemAction] Don't offer for non-local URLs

Elvis Angelaccio noreply at phabricator.kde.org
Tue Aug 14 22:11:45 BST 2018


elvisangelaccio requested changes to this revision.
elvisangelaccio added a comment.
This revision now requires changes to proceed.


  Looks good otherwise.

INLINE COMMENTS

> compressfileitemaction.cpp:53
> +    const auto urlList = fileItemInfos.urlList();
> +    const bool hasLocalUrl = std::any_of(urlList.begin(), urlList.end(), [](const QUrl &url) {
> +        return url.isLocalFile();

`constBegin()` / `constEnd()`

REPOSITORY
  R36 Ark

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

To: broulik, elvisangelaccio, dfaure
Cc: kde-utils-devel, #ark, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20180814/def41787/attachment.html>


More information about the Kde-utils-devel mailing list