<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 16, 2014 at 5:32 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank" class="cremed">thakis@chromium.org</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 class="gmail_extra"><div class="gmail_quote"><div class="">On Mon, Jun 16, 2014 at 6:29 AM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank" class="cremed">djasper@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"><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" target="_blank" class="cremed">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" target="_blank" class="cremed">reviews.llvm.org</a>. If at all possible, I'd like to do the actual code review there.</div>

</div></blockquote><div><br></div></div><div>As far as I know, sending patches via email is still one of the supported workflows and mentioned on the web sites. Did I miss that this changed?</div></div></div></div></blockquote>
<div><br></div><div>Well, we (the key contributors to clang-format) strongly prefer the code review system (both because it is easier and because we have spent quite a bit of time setting it up). So, clang-format patches are preferred to be sent there.</div>
<div> </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"><div class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">
<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"><div><div>On Mon, Jun 16, 2014 at 2:30 PM, Sobczak, Janusz <span dir="ltr"><<a href="mailto:janusz.sobczak@mobica.com" target="_blank" class="cremed">janusz.sobczak@mobica.com</a>></span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><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></div></div>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div></div><br></div></div>
</blockquote></div><br></div></div>