Differential D28415

Archives: Rename three KConfigGroups to make things clearer
ClosedPublic
Actions

Authored by asensi on Mar 30 2020, 5:46 AM.

Details

Reviewers
None
Group Reviewers
Krusader
Maniphest Tasks
T12876: Solve some bugs related to archives, remove some duplicated code
Commits
R167:8277f9f17dc4: Archives: Rename three KConfigGroups to make things clearer
Summary

Rename two KConfigGroups named confGrp to dependGrp, and a KConfigGroup named config to dependGrp, because they only operate with the "Dependencies" group. That also easies keeping code synchronized.

Test Plan

The test plan of the task defined in Solve some bugs related to archives, remove some duplicated code.

Diff Detail

Repository
R167 Krusader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
asensi requested review of this revision.Mar 30 2020, 5:46 AM
asensi created this revision.
asensi edited the summary of this revision. (Show Details)Mar 30 2020, 8:37 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 15 2020, 10:46 PM
Closed by commit R167:8277f9f17dc4: Archives: Rename three KConfigGroups to make things clearer (authored by asensi). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
MkrArc/krarc.h (2 lines)
MkrArc/krarc.cpp (6 lines)
Mkrusader/krservices.cpp (12 lines)

Diff 80245

View Options

krArc/krarc.h

Loading...
View Options

krArc/krarc.cpp

Loading...
View Options

krusader/krservices.cpp

Loading...