<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2016-12-12 19:20 GMT+01:00 Eugene Zelenko <span dir="ltr"><<a href="mailto:eugene.zelenko@gmail.com" target="_blank">eugene.zelenko@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Cześć, Piotr!<br>
<div><div class="gmail-h5"><br>
On Mon, Dec 12, 2016 at 7:02 AM, Piotr Padlewski<br>
<<a href="mailto:piotr.padlewski@gmail.com">piotr.padlewski@gmail.com</a>> wrote:<br>
> Hi,<br>
> as clang-tidy grows there are more and more checks. One of the problem I see<br>
> is that "misc" check group is not user friendly - there are many checks that<br>
> do so many different things that usually user don't want to enable whole<br>
> group.<br>
> Other groups like modernize, performance, google, cert, boost, llvm doesn't<br>
> have this problem. Naturally the solution would be to split the group into<br>
> smaller groups that would mean more.<br>
> The problem is that we should not change names because old configs will not<br>
> work.<br>
><br>
> Do you have some ideas how we could fix it, so we could make it easier for<br>
> users to use it?<br>
><br>
> Other feature that we could add if we would know how to solve it is that we<br>
> could make new groups that would mostly have aliases to other checks. This<br>
> might be specially useful for cert checks - the cert code names doesn't tell<br>
> anything, so it would be good to have these checks with proper name in<br>
> different group so normal user could see what this check is doing from name<br>
> and CERT users could run checks with cert group as it was before.<br>
><br>
><br>
> One solution that I see is to reserve old name and make redirection, and<br>
> maybe output warning about deprecated name when user would use special flag<br>
> (e.g. verbose)<br>
><br>
> What do you think about this problem?<br>
><br>
> Piotr<br>
<br>
</div></div>It's really good idea to split misc checks!<br>
<br>
My suggestions:<br>
<br>
I think it’ll be good idea to move unused checks to Clang, but will<br>
this be easy? Clang has warnings for unused variables, static<br>
functions and macros already. GCC has warning for unused parameters.<br>
<br>
misc-unused-alias-decls<br>
misc-unused-parameters<br>
misc-unused-raii<br>
misc-unused-using-decls<br>
<br></blockquote><div>Or maybe it should be in one check with options? Maybe only misc-unused-parameters is different enough to have special option to disable it.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
misc-inefficient-algorithm should be moved to performance. I tried to<br>
do this in past, but my patch was not perfect.<br>
<br>
Probable we should create next check groups:<br>
<br>
Potential memory issues:<br>
<br>
misc-use-after-move<br>
<br>
Preprocessor:<br>
<br>
misc-macro-parentheses<br>
misc-macro-repeated-side-<wbr>effects<br>
misc-multiple-statement-macro<br>
<br>
Casts:<br>
<br>
misc-bool-pointer-implicit-<wbr>conversion<br>
misc-incorrect-roundings<br>
misc-misplaced-widening-cast<br>
<span class="gmail-HOEnZb"><font color="#888888"><br></font></span></blockquote><div>The reset sounds good!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-HOEnZb"><font color="#888888">
Eugene.<br>
</font></span></blockquote></div><br></div></div>