Differential D7737

[KateConfigDialog] Don't iterate values()
ClosedPublic
Actions

Authored by broulik on Sep 8 2017, 2:25 PM.

Details

Reviewers
kfunk
Group Reviewers
Kate
Commits
R40:b767a96e1d58: [KateConfigDialog] Don't iterate values()
Summary

Foreach does exactly that without the need to create a temporary list first.

Test Plan

Stuff is still applied

Diff Detail

Repository
R40 Kate
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Sep 8 2017, 2:25 PM
Restricted Application added a subscriber: kwrite-devel. · View Herald TranscriptSep 8 2017, 2:25 PM
broulik edited reviewers, added: Kate; removed: katew.Sep 8 2017, 2:25 PM
kfunk accepted this revision.Sep 8 2017, 8:56 PM
This revision is now accepted and ready to land.Sep 8 2017, 8:56 PM
Closed by commit R40:b767a96e1d58: [KateConfigDialog] Don't iterate values() (authored by broulik). · Explain WhySep 14 2017, 10:20 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkate/kateconfigdialog.cpp (4 lines)

Diff 19516

View Options

kate/kateconfigdialog.cpp

Loading...