D8414: libzip: Add support for compression methods and levels

Elvis Angelaccio noreply at phabricator.kde.org
Sun Oct 22 09:20:46 UTC 2017


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

INLINE COMMENTS

> libzipplugin.cpp:275
> +    }
> +    int compLevel = 6;
> +    if (options.isCompressionLevelSet()) {

Can be simplified in:

`const int compLevel = options.isCompressionLevelSet() ? options.compressionLevel() : 6;`

REPOSITORY
  R36 Ark

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

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/20171022/4b56c3a8/attachment.html>


More information about the Kde-utils-devel mailing list