Differential D19317

compile without foreach
ClosedPublic
Actions

Authored by mlaurent on Feb 25 2019, 8:45 PM.

Details

Reviewers
dfaure
apol
Commits
R287:f1c6c15b061b: compile without foreach
Summary

compile without foreach

Diff Detail

Repository
R287 KImageFormats
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.Feb 25 2019, 8:45 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 25 2019, 8:45 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
mlaurent requested review of this revision.Feb 25 2019, 8:45 PM
apol added a subscriber: apol.Feb 26 2019, 2:51 PM
apol added inline comments.
src/imageformats/pic.cpp
391–392

Probably could do a splitRef here

tests/imageconverter.cpp
67–68

I'd use auto in this cases, otherwise you're creating a dependency to the type.

mlaurent updated this revision to Diff 52700.Feb 27 2019, 5:58 AM

Use auto as requested

mlaurent added inline comments.Feb 27 2019, 5:58 AM
src/imageformats/pic.cpp
391–392

it seems that we can't use it.
As it will return a list of QStringRef
and we want to use "simplified() on a QStringRef which doesn't exist

apol accepted this revision.Feb 28 2019, 4:25 PM
apol added inline comments.
tests/imagedump.cpp
75–76

auto too?

This revision is now accepted and ready to land.Feb 28 2019, 4:25 PM
Closed by commit R287:f1c6c15b061b: compile without foreach (authored by mlaurent). · Explain WhyFeb 28 2019, 10:02 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 52863

View Options

CMakeLists.txt

Loading...
View Options

autotests/readtest.cpp

Loading...
View Options

autotests/writetest.cpp

Loading...
View Options

src/imageformats/pic.cpp

Loading...
View Options

tests/imageconverter.cpp

Loading...
View Options

tests/imagedump.cpp

Loading...