Port endl to "\n" flush at the end + use const'ref in loop
ClosedPublic

Authored by mlaurent on Dec 31 2019, 6:03 AM.

Diff Detail

Repository
R266 Breeze Icons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mlaurent created this revision.Dec 31 2019, 6:03 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 31 2019, 6:03 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
mlaurent requested review of this revision.Dec 31 2019, 6:03 AM
apol added a subscriber: apol.Dec 31 2019, 6:50 PM
apol added inline comments.
autotests/scalabletest.cpp
312–316

'\n' or merge into the string.

319

Maybe add a {}-scope?

mlaurent updated this revision to Diff 72513.Jan 1 2020, 9:41 AM

merge \n or add '\n'

apol accepted this revision.Jan 1 2020, 11:34 PM
This revision is now accepted and ready to land.Jan 1 2020, 11:34 PM
This revision was automatically updated to reflect the committed changes.