<div dir="ltr"><div>Thank you for working on this. This was started and abandoned (?) before:</div><div><a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20131007/090583.html">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20131007/090583.html</a><br>
</div><div>(It is quite old, but might be worth reading up on, I had a few comments).<br></div><div><br></div><div>As for this patch:</div><div>- We now have a code review system at <a href="http://reviews.llvm.org">reviews.llvm.org</a>. If at all possible, I'd like to do the actual code review there.</div>
<div>- At some stage we discussed that it really is "PointerAlignment" and not "PointerBinding", so if we are renaming this already, lets fix this as well.</div><div>- We should try to make this backwards compatible, so clang-format should still accept the old settings in the configuration file. For an example of a bool that was changed to an enum see UseTabStyle. I am not quite sure how to map two different configuration names ("PointerBindsToType" and "PointerAlignment") to the same variable, but I am happy to do the renaming as a follow up patch if you can't figure it out.</div>
<div>- We need more tests, copy more of the .."UsesOfStarAndAmp" tests.</div><div>- The enumerator names aren't really ideal as there isn't always a "variable". I liked the "Left", "Right" and "Middle" from the other patch.</div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 16, 2014 at 2:30 PM, Sobczak, Janusz <span dir="ltr"><<a href="mailto:janusz.sobczak@mobica.com" target="_blank">janusz.sobczak@mobica.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"><div><div><div><div>Hi<br><br>The patch implements 3 types of pointer bindings:<br></div>type *var;<br></div>
type* var;<br></div>type * var;<br><br></div>Feature requested in bug 19929.<br clear="all"><div><div>
<div><div><div><br></div><div>Cheers,<br></div><div>Janusz<br>
</div></div></div></div></div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">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/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>