<div dir="ltr">I believe this patch will take a couple more steps to 'clangify' it.<div>Can you submit it via the normal patch review process to cfe-commits (optimally via phabricator, so the code review is easier)</div><div><br></div><div>Thanks!</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 23, 2015 at 9:06 PM Sterling Augustine <<a href="mailto:saugustine@google.com">saugustine@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 23, 2015 at 2:07 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@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 dir="ltr"><p dir="ltr">The right next step is to send a patch / code review where we can discuss the change. Is there one that I have missed?</p></div></blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>This required a bit of  editing, so I've now trimmed it to just the portions that I need. Which makes the patch quite a bit smaller. So there's that.</div><div><br></div><div>Enclosed. It is likely I missed some stylistic issues, but clang-format -llvm has done some magic too.</div><div><br></div><div>Let me know the next steps.</div><div><br></div><div>Also, I want to be sure the Cling project is OK with the way it is attributed. Happy to work with you all however you like there.</div><div><br></div><div><br></div></div></div></div>
</blockquote></div>