> On Aug. 24, 2016, 10:04 p.m., Benjamin Mahler wrote:
> > Ship It!
Note that the change isn't C++11 related (my TODO was suggesting to use a
lambda which requires C++11, but your change just uses defer. Since this didn't
use to work, I updated the commit summary to be the following:
```
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50853/#review146720
---
Ship it!
Ship It!
- Benjamin Mahler
On Aug. 5, 2016, 3:30
> On Aug. 8, 2016, 6:45 a.m., Benjamin Mahler wrote:
> > Which compilers did you test with?
I tested with:
`gcc 4.8.1, 4.8.4, 4.8.5, 4.9.2, 5.3.1`
`clang 7.0.2`
- Gastón
---
This is an automatically generated e-mail. To reply, visit:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50853/#review145061
---
Which compilers did you test with?
- Benjamin Mahler
On Aug.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50853/#review144974
---
Patch looks great!
Reviews applied: [50853]
Passed command:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50853/
---
Review request for mesos, Alexander Rukletsov and Benjamin Mahler.
Repository:
