<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 8, 2015 at 1:37 AM, Francis Visoiu Mistrih via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@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">Hello Philip,<br>
<span class=""><br>
The 12/07/2015 10:43, Philip Reames wrote:<br>
> I think this patch - and the others of similar vein you've posted - are flat<br>
> out a bad idea.  They diminish readability for no gain.  Do you have a good<br>
> argument for why unnamed parameters are a good idea?<br>
</span>The unused parameters cause warinings [-Wunused-parameter].<br></blockquote><div><br></div><div>But we don't have that warning enabled, do we? (at least the Clang self-hosting build doesn't seem to ever trigger it, so I assume we don't have it enabled)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Would commenting the parameter names, or casting them to void in the<br>
body of the function be a better idea?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Francis Visoiu Mistrih<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>