You should wait for an explicit ack on committing a patch if it's in review. It's a bit anti-social to do so otherwise and that people are waiting isn't a good enough reason to skip that.<br><div><br></div><div>
In addition, asking you to improve the comments was a way of getting a better explanation of the code without having it tied up in email - if I have to ask then so would someone else. I may have other feedback for you after the comments are added.</div>
<div><br></div><div>-eric</div><br><div>On Mon Jan 06 2014 at 2:36:28 PM, Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Justin Bogner <<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>> writes:<br>
> Since we're basically just improving comments at this point, and since<br>
> this not being in has been blocking other people trying to use and<br>
> build on top of these changes, I'm going to go ahead and commit<br>
> now. I'll continue to incorporate further feedback in follow on<br>
> commits.<br>
<br>
r198638, r198639, and r198640.<br>
______________________________<u></u>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><br>
</blockquote>