<div dir="ltr">On Wed, Jan 23, 2013 at 11:53 AM, Dmitri Gribenko <span dir="ltr"><<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Jan 15, 2013 at 7:31 PM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:<br>

> On Tue, Jan 8, 2013 at 8:38 PM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>> On 8 January 2013 11:31, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:<br>
>>> Hello Rafael,<br>
>>><br>
>>> I have added the requested tests.<br>
>><br>
>> Cool. The patch look good to me. Just give chandler a day or so to<br>
>> comment on it. The google style bans VLAs, so he might have an opinion<br>
>> on the patch.<br>
><br>
> Ping.<br>
<br>
</div>Ping^2.<br>
<br>
Chandler: do you have concerns about this?</blockquote><div><br></div><div style>Sorry, I missed Rafael's comment. Google's style should have no bearing on a warning. That doesn't make any sense.</div><div style>
<br></div><div style>This patch makes a lot of sense. I don't know what could be bad about it.</div><div style><br></div><div style>I assume the idea of warning about vla in general is for GCC compatibility or security concerns? Seems reasonabl eto keep that functionality.</div>
<div style><br></div><div style>Anyways, don't wait for me, commit! </div></div></div></div>