<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Mar 1, 2016 at 10:10 AM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg requested changes to this revision.<br>
clayborg added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
One general comment is the use of "auto". Although it makes the code shorter, it does make it quite a bit less readable. I will leave the decision to you since this is your code, but in general I think this is where auto is less than it is cracked up to be.<br></blockquote><div>I agree with you in principle, but I used it here because these types are really ugly because they're unique pointers of types whose names are already long.  I could typedef them to UP or something, but they wouldn't be able to go in lldb-forward.h because they shouldnt be publicly exposed.  I'll think about it some more and decide what to do.</div><div><br></div><div>Have to run right now, will respond to the rest of your comments later.</div><div> </div></div></div>