Differential D24584

Add a few const &
ClosedPublic
Actions

Authored by aacid on Oct 12 2019, 8:39 AM.

Details

Reviewers
meven
Commits
R293:add9ec4f72cd: Add a few const &
Summary

These things aren't super expensive to copy, but const & is less expensive

Diff Detail

Repository
R293 Baloo
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17603
Build 17621: arc lint + arc unit
aacid created this revision.Oct 12 2019, 8:39 AM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptOct 12 2019, 8:39 AM
Restricted Application added subscribers: Baloo, kde-frameworks-devel. · View Herald Transcript
aacid requested review of this revision.Oct 12 2019, 8:39 AM
meven accepted this revision.Oct 12 2019, 9:31 AM
meven added a subscriber: meven.

Seems good to me

This revision is now accepted and ready to land.Oct 12 2019, 9:31 AM
aacid closed this revision.Oct 12 2019, 10:04 PM

Revision Contents
Changeset List

Diff 67773

View Options

src/engine/idtreedb.cpp

Loading...
View Options

src/engine/writetransaction.h

Loading...
View Options

src/engine/writetransaction.cpp

Loading...
View Options

src/kioslaves/search/kio_search.h

Loading...
View Options

src/kioslaves/search/kio_search.cpp

Loading...