-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27945/#review61291
-----------------------------------------------------------

Ship it!


Thanks Cody!

Please end the review summaries with periods, as they get directly translated 
into commit messages, I updated the message:

"Added c++11 initializer_list check to configure."

We've tried to use past tense but present tense has been slipping through a lot 
as of late. ;)


m4/ax_cxx_compile_stdcxx_11.m4
<https://reviews.apache.org/r/27945/#comment102833>

    Let's match the brace-on-a-newline style here :)
    
    I'll do this for you.



m4/ax_cxx_compile_stdcxx_11.m4
<https://reviews.apache.org/r/27945/#comment102837>

    Let's remove the extra whitespace introduced here, I'll do this for you.


- Ben Mahler


On Nov. 13, 2014, 1:02 a.m., Cody Maloney wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27945/
> -----------------------------------------------------------
> 
> (Updated Nov. 13, 2014, 1:02 a.m.)
> 
> 
> Review request for mesos, Ben Mahler and Dominic Hamon.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Check both that initializer_list exists, and that a standard container 
> (vector) accepts it.
> 
> 
> Diffs
> -----
> 
>   m4/ax_cxx_compile_stdcxx_11.m4 d166aa1adf56888ae8f6025469e35165e1f21d61 
> 
> Diff: https://reviews.apache.org/r/27945/diff/
> 
> 
> Testing
> -------
> 
> make distcheck with GCC on ubuntu 14.04
> 
> 
> Thanks,
> 
> Cody Maloney
> 
>

Reply via email to