<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 9, 2012 at 6:24 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>A couple of things:</div>* AttributedStmt::CreateEmpty isn't allocating enough space for the Attr*s.<br></blockquote>
<div>Oops, my bad. Fixed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">* Please capitalize the parameters to the AttributedStmt member functions.<br>
</blockquote><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM with those fixed.</blockquote><div>Thanks for the review, committed as rev. 159925.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br><br>
<div class="gmail_quote">On Fri, Jul 6, 2012 at 3:24 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Did anyone have time to look at this?<div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 5, 2012 at 9:42 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<div><br></div><div>This patch changes AttrVec to inline storage of attributes in AttributedStmt. I also had to add a template parameter for container type to specific_attr_iterator to make it handle ArrayRef along with AttrVec. Updated serialization code, but there's still no good way to test it.</div>



<div><br></div><div>Please take a look at this patch.</div><span><font color="#888888"><div><div><br></div>-- <br>
<div>Regards,</div><div>Alexander Kornienko</div></div>
</font></span></blockquote></div><br>
</div>
</div></div></blockquote></div><br>
</div></div></blockquote></div><div><br></div>-- <br>
</div><div class="gmail_extra">Regards,</div><div class="gmail_extra">Alexander</div>