D20224: Implement saving sessions recursively

[Tomaz Canabrava]

tcanabrava added inline comments.

INLINE COMMENTS

> hindenburg wrote in ViewManager.cpp:910
> FYI, loop variable 'widgetJsonValue' is always a copy because the range of type 'QJsonArray' does not return a reference

Having the reference of a temporary is not a problem, and considering the clazy warning on your other comment I belive this is the way to go.

> hindenburg wrote in ViewManager.cpp:931
> Missing reference in range-for with non trivial type (QJsonValue) [-Wclazy-range-loop

That's a QJsonArray too. I'll just add the reference to the temporary.

REPOSITORY
  R319 Konsole

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

To: tcanabrava, #konsole, hindenburg, ngraham
Cc: konsole-devel, gennad, thsurrel, ngraham, maximilianocuria, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20190409/e7da75aa/attachment.html>