Maybe include boost::adaptor into stout under namespace adaptor? I liked the adaptor idea since it can be composed.
- Jie On Sat, Jan 16, 2016 at 10:48 AM, Benjamin Mahler <[email protected]> wrote: > It seems unfortunate to introduce alternative looping constructs, for > example the following would be the more composable approach: > > foreach (int i, reversed(numbers)) { > > } > > I remember this coming up before: > > http://mail-archives.apache.org/mod_mbox/mesos-dev/201410.mbox/%[email protected]%3E > > Thoughts? > > On Saturday, January 16, 2016, Jie Yu <[email protected]> wrote: > >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/42379/#review114871 >> ----------------------------------------------------------- >> >> Ship it! >> >> >> >> 3rdparty/libprocess/3rdparty/stout/include/stout/foreach.hpp (lines 55 - >> 58) >> <https://reviews.apache.org/r/42379/#comment175706> >> >> Ca you move this below 'foreachvalue'? >> >> >> - Jie Yu >> >> >> On Jan. 16, 2016, 1:32 a.m., Timothy Chen wrote: >> > >> > ----------------------------------------------------------- >> > This is an automatically generated e-mail. To reply, visit: >> > https://reviews.apache.org/r/42379/ >> > ----------------------------------------------------------- >> > >> > (Updated Jan. 16, 2016, 1:32 a.m.) >> > >> > >> > Review request for mesos and Jie Yu. >> > >> > >> > Repository: mesos >> > >> > >> > Description >> > ------- >> > >> > Add reverse_foreach in libprocess. >> > >> > >> > Diffs >> > ----- >> > >> > 3rdparty/libprocess/3rdparty/stout/include/stout/foreach.hpp >> 7fb0044790ee249b69e07b81a26851bd5bfb110f >> > >> > Diff: https://reviews.apache.org/r/42379/diff/ >> > >> > >> > Testing >> > ------- >> > >> > make check >> > >> > >> > Thanks, >> > >> > Timothy Chen >> > >> > >> >>
