We should also come up with the recommended way of dealing with foreachpair and foreachvalue occurrences. I did a quick grep and there seem to be over 900 places where foreach(value/pair) are used. It's going to be quite a change (albeit a straightforward one), but we should still be all for it :-).
Cheers, Artem. On Tue, Aug 4, 2015 at 5:18 PM, Benjamin Mahler <[email protected]> wrote: > I noticed there was a single ranged-based for loop introduced in the source > tree: > > https://github.com/apache/mesos/blob/master/src/tests/resources_tests.cpp#L1004 > > Since this is the only one in the codebase AFAICT, I've removed it to keep > our use of 'foreach' consistent. Michael, is there a plan to replace > 'foreach' with range-based for loops? Would just like there to be a > consistent way to loop over containers. :)
