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