<br><div class="gmail_quote">On Thu, Aug 30, 2012 at 3:06 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im">On Thu, Aug 30, 2012 at 1:06 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">

Looks nice!<div><br>I'm not the one to approve it, but here are few minor comments regarding the patch (btw, it would be much more convenient with <a href="http://llvm-reviews.chandlerc.com/" target="_blank">http://llvm-reviews.chandlerc.com/</a></div>

</blockquote></div><div>I haven't been paying attention to the review options for this mailing list.  I'll try sending future versions of this patch there.</div></div></blockquote><div>I missed the "this is a work in progress" part ;) </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div> ;):</div></blockquote><div> <a href="http://xkcd.com/541/" target="_blank">http://xkcd.com/541/</a></div></div></blockquote><div>;)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>Besides that, are you going to deal with AST dumping further? I was planning to start replacing current Decl dumping with something more informative (based on current -ast-dump-xml implementation, probably) at some point. It would be nice to try to avoid conflicts ;)</div>


<div><br></div></div></blockquote></div><div>I did notice the FIXME's in DumpDeclarator().  I'm aware of it, but haven't started any work on it.  I'll ping you if I start working in that direction.  Are you considering starting work on it soon?</div>
</div></blockquote><div>It could be next week, but I'm not sure yet. I can safely delay this work until you're done with your changes. </div><div><br></div></div><br clear="all"><div>-- </div><div>Regards,</div><div>
Alex</div>