Please turn it on in your browser preferences.
for specific sites, we recommend the
Review Request #129378 -
Created Nov. 11, 2016 and submitted
Aug. 24, 2017, 5:06 a.m.
No functional changes, just adding some const & to avoid copies where possible.
I don't think this is saving anything, the move constructor should be taking care of it.
I'm sorry but where do you see move ctors here in the first place? it.next() is returning a simple const ref which must not be modified. How should this data moved from this const iterator to the local object?
Ping, no interest anyone?
Anyone interested? Otherwise I'll discard this review in the next days
Submitted with commit a75db7aedfcd356b027774e0dc156c334d4cb869 by Christian Ehrlicher to branch master.