Differential D1251

Update replacement of ListArgs parameters
ClosedPublic
Actions

Authored by elvisangelaccio on Mar 29 2016, 2:04 PM.

Details

Reviewers
rthomsen
Commits
R36:482e771859ff: cliinterface: refactor the replacement of ListArgs
Summary

Same thing we did for the ExtractArg ones.

Test Plan

The new function is unit tested. We can add tests for clizip once we merge D1249

Diff Detail

Repository
R36 Ark
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
elvisangelaccio updated this revision to Diff 3003.Mar 29 2016, 2:04 PM
elvisangelaccio retitled this revision from to Update replacement of ListArgs parameters.
elvisangelaccio updated this object.
elvisangelaccio edited the test plan for this revision. (Show Details)
elvisangelaccio added a reviewer: rthomsen.
elvisangelaccio set the repository for this revision to R36 Ark.
elvisangelaccio added a project: Ark.
elvisangelaccio added a subscriber: kde-utils-devel.
rthomsen accepted this revision.Mar 30 2016, 12:33 PM
rthomsen edited edge metadata.

Looks good :)

This revision is now accepted and ready to land.Mar 30 2016, 12:33 PM

Revision Contents
Changeset List

Diff 3026

View Options

autotests/plugins/cli7zplugin/cli7ztest.h

Loading...
View Options

autotests/plugins/cli7zplugin/cli7ztest.cpp

Loading...
View Options

autotests/plugins/clirarplugin/clirartest.h

Loading...
View Options

autotests/plugins/clirarplugin/clirartest.cpp

Loading...
View Options

autotests/plugins/cliunarchiverplugin/cliunarchivertest.h

Loading...
View Options

autotests/plugins/cliunarchiverplugin/cliunarchivertest.cpp

Loading...
View Options

kerfuffle/cliinterface.h

Loading...
View Options

kerfuffle/cliinterface.cpp

Loading...
View Options

plugins/cliunarchiverplugin/cliplugin.cpp

Loading...