[Kfilemetada] Make it compile without foreach
AbandonedPublic

Authored by mlaurent on Mar 1 2019, 12:14 PM.

Details

Summary

compile without foreach

Test Plan

Autotest ok

Diff Detail

Repository
R286 KFileMetaData
Branch
compile_without_foreach (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9019
Build 9037: arc lint + arc unit
mlaurent created this revision.Mar 1 2019, 12:14 PM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptMar 1 2019, 12:14 PM
Restricted Application added subscribers: Baloo, kde-frameworks-devel. · View Herald Transcript
mlaurent requested review of this revision.Mar 1 2019, 12:14 PM
apol accepted this revision.Mar 1 2019, 12:26 PM
This revision is now accepted and ready to land.Mar 1 2019, 12:26 PM

There are currently larger refactorings ongoing (D19287, D18826), could you please wait for them and rebase afterwards?

src/externalextractor.cpp
151

missing space: for (

src/extractorplugin.cpp
140

missing space: for (

bruns requested changes to this revision.Mar 1 2019, 6:42 PM
bruns added a subscriber: bruns.

This clashes with D19287, and has to be rebased/merged then

This revision now requires changes to proceed.Mar 1 2019, 6:42 PM
This revision was not accepted when it landed; it landed in state Needs Revision.Mar 4 2019, 6:03 AM
This revision was automatically updated to reflect the committed changes.

Arg it was commited by error
I reverted it

bruns reopened this revision.Mar 9 2019, 12:28 AM

Can be rebased/merged now ...