<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2018-03-20 22:03 GMT+01:00 Richard via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">[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>
In article <<a href="mailto:BN6PR13MB1569F3CBF08B69AED1DEC207E8D40@BN6PR13MB1569.namprd13.prod.outlook.com">BN6PR13MB1569F3CBF08B69AED1DE<wbr>C207E8D40@BN6PR13MB1569.<wbr>namprd13.prod.outlook.com</a>>,<br>
<span class="">    "Krzemienski, Andrzej via cfe-dev" <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> writes:<br>
<br>
<br>
> It would be too much false positives to NOLINT.<br>
<br>
</span>It seems that you have to do the edits to add the attribute, so how is<br>
that different from doing the edits to add NOLINT?<br></blockquote><div><br></div><div>The way I see the difference is that putting a NOLINT is an indication of either:<br></div><div>1. An inaccurate check (it has false positives that I have to silence)<br></div><div>2. My inability to fix my code (because I am not in control of it, or because I have no time to investigate)<br><br></div><div>Silencing the warning with a dedicated attribute is saying: the check is accurate, my code is correct and clearly states intentions.<br><br></div><div>And there is still this other point. I only want to check for inadvertent conversions caused by programmer forgetting to declare her constructor explicit. I *do not want* to warn on non-explicit conversions operators: they are never added by omission.<br></div><div>If I go with google-explicit-<wbr>constructor I will have to NOLINT every non-explicit conversion operator. If I go with the proposed check (which does not check conversion operators -- only constructors) I do not have to NOLINT any conversion operator.<br><br></div><div>Does this make sense?<br><br></div><div>Regards,<br></div><div>&rzej;<br></div></div><br></div></div>