<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 10, 2015 at 9:29 AM, Richard via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
[Please reply *only* to the list and do not include my email directly<br>
in the To: or Cc: of your reply; otherwise I will not see your reply.<br>
Thanks.]<br>
<br>
</span>The comments on this check seem to be asking for the *reverse* check:<br>
<br>
switch everything to use auto<br></blockquote><div><br></div><div>Commenting from the sidelines, I see two things:</div><div>1. For more "traditional" (for lack of a better term) C++ codebases, removing "unnecessary" uses of the -> syntax is probably a worthwhile thing on the grounds of consistency (we would probably want something like that in LLVM, not that we seem to have that problem in LLVM). I.e., the policy is roughly "use the -> syntax only where strictly necessary, but otherwise, for consistency, use the traditional syntax"</div><div>2. The `auto f() -> T` syntax opens up a new style altogether, and green pasture projects might want to uniformly use that style.</div><div><br></div><div>So I think your original post is basically about 1. and some commenters have noted the existence of 2.</div><div><br></div><div>They're basically separate use cases and I don't think one should hold up the other as far as writing clang-tidy checks. Do you know of any projects that are having issues with -> being used "unnecessarily"?</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Yes?<br>
<span class="im HOEnZb">--<br>
"The Direct3D Graphics Pipeline" free book <<a href="http://tinyurl.com/d3d-pipeline" rel="noreferrer" target="_blank">http://tinyurl.com/d3d-pipeline</a>><br>
     The Computer Graphics Museum <<a href="http://ComputerGraphicsMuseum.org" rel="noreferrer" target="_blank">http://ComputerGraphicsMuseum.org</a>><br>
         The Terminals Wiki <<a href="http://terminals.classiccmp.org" rel="noreferrer" target="_blank">http://terminals.classiccmp.org</a>><br>
  Legalize Adulthood! (my blog) <<a href="http://LegalizeAdulthood.wordpress.com" rel="noreferrer" target="_blank">http://LegalizeAdulthood.wordpress.com</a>><br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div></div>