D5359: cliunarchiver: Emit encryption method

Elvis Angelaccio noreply at phabricator.kde.org
Sun Apr 9 15:40:43 UTC 2017


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

INLINE COMMENTS

> cliplugin.cpp:264
>          currentEntry->setProperty("size", currentEntryJson.value(QStringLiteral("XADFileSize")));
> -        currentEntry->setProperty("isPasswordProtected", (currentEntryJson.value(QStringLiteral("XADIsEncrypted")).toInt() == 1));
> +        bool m_isPasswordProtected = (currentEntryJson.value(QStringLiteral("XADIsEncrypted")).toInt() == 1);
> +        currentEntry->setProperty("isPasswordProtected", m_isPasswordProtected);

const bool, and remove the m_ prefix (not a class member variable).

REPOSITORY
  R36 Ark

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

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


More information about the Kde-utils-devel mailing list