D23597: Bulk port away from foreach

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Sun Sep 1 07:50:44 BST 2019


kossebau added inline comments.

INLINE COMMENTS

> kossebau wrote in kfontsizeaction.cpp:93
> Will test is this works, but IIRC actions() returned a normal left-side value thingie, which qAsConst does not want to take.

Yes, `qAsConst(actions())` does not work, as actions() is a r-value type, which qAsConst does not take (move overload of qAsConst explicitly deleted), following std::as_const here, to not run into dangling pointers.

REPOSITORY
  R236 KWidgetsAddons

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

To: kossebau, #frameworks, cfeck
Cc: dhaumann, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190901/0da19cf6/attachment.html>


More information about the Kde-frameworks-devel mailing list