<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 25, 2014 at 7:17 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
On 25/06/2014 14:06, Alexander Kornienko wrote:<div class=""><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On Wed, Jun 25, 2014 at 10:48 AM, Alp Toker <<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a> <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>>> wrote:<br>
<br>
    How about just using -Wold-style-cast?<br>
<br>
<br>
I thought, it was agreed upon, that the -Wold-style-cast warning isn't going to suggest automatic fixes: <a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20131125/094121.html" target="_blank">http://lists.cs.uiuc.edu/<u></u>pipermail/cfe-commits/Week-of-<u></u>Mon-20131125/094121.html</a>. However, this check is going to.<br>

<br>
Any other concerns?<br>
<br>
</blockquote>
<br></div>
Hmm, if this is identical to -Wold-style-cast why is it going into clang-tidy/google/?<br>
<br>
There doesn't seem to be anything Google-specific at all here, any more than the equivalent compiler flag in clang is Nuanti-specific.</blockquote><div><br></div><div>Currently, we're trying to organize the checks by coding style to make it easy to select all checks from a certain style (using -checks=-*,google-* or -checks=-*,llvm-*). And from the two coding styles we are currently planning to support, only one specifically has the rule to avoid C-style casts. If the check didn't implement a rule of one of this styles, it would go misc/.</div>
<div><br></div><div>We know that this system is sub-optimal, as categorization of the checks by style doesn't work well when checks can be shared between styles. A better approach could be to have checks arranged separately by some property (e.g. some characteristic of the issue they address - readability/performance/compatibility/bug-prone coding patterns), and the styles being just lists of checks with specific configurations and meta-data (e.g. links to specific style guide rules).</div>
<div><br></div><div>We're going to do this kind of rearrangement, but it will be done independently from this patch.</div></div>
</div></div>