dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


  Looks like this got lost/abandoned? It's painful to review (because it's so 
long), but it shouldn't be lost work...
  
  Can you rebase and see if it still applies?
  
  Maybe in the future better split this up into multiple patches so it can land 
in chunks instead of getting stuck forever....

INLINE COMMENTS

> broulik wrote in daysmodel.cpp:171
> While at it `const QDate &`

Actually better not, a QDate is just a wrapper for a qint64, with a generated 
copy constructor (so it's just "copying" a qint64)

> corona.cpp:467
>      KConfigGroup containmentsGroup(cg, "Containments");
> -    foreach (const Containment *containment, containments) {
> +    for (const Containment *containment : qAsConst(containments)) {
>          QString cid = QString::number(containment->id());

(already const, this method is const)

> apol wrote in pluginloader.cpp:567
> shouldn't it be const?

(it is now)

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  compile_without_foreach (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D19913

To: mlaurent, dfaure
Cc: broulik, apol, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, 
bruns

Reply via email to