<div dir="ltr">Yep, that's the general idea - the bar for Clang warnings has generally (with some variation) been to have a low false-positive rate (where a false-positive is "warns the user even though there is no behavioral bug (or significant risk of confusion from readers of the source code) in the code") - so a blanket warning on all missing overrides wouldn't meet that bar - lots of code gets along fine without them. But if the user wrote at least one 'override' keyword in a class, fair bet they meant to write them all, so warning in that case seems feasible.<br><br>clang-tidy is more stylistic & has a lower bar (one of the reasons for the different bar is that clang-tidy's warnings don't impact the clang codebase in both code complexity and performance - which is a reason we can't implement all the possible warnings in there).<br><br>- Dave<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 10, 2018 at 2:23 AM Jonas Toth via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <p>Hi,</p>
    <p>I experience the override-warning when it was inconsistent within
      the class (meaning it has been used at least once and some places
      missed it). This measure is probably to not warn on old code
      bases.</p>
    <p>Could this be the case?</p>
    <p>Best, Jonas<br>
    </p></div><div text="#000000" bgcolor="#FFFFFF">
    <br>
    <div class="m_4749730040338246124moz-cite-prefix">Am 09.08.2018 um 19:26 schrieb JVApen
      via cfe-dev:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="auto">Hello all,
        <div dir="auto"><br>
        </div>
        <div dir="auto">Today we have discovered a case where we did not
          have a compiler warning for a missing override. To verify, we
          explicitly enabled the warning with the pragma.</div>
        <div dir="auto"><br>
        </div>
        <div dir="auto">While investigating this in the IDE, we noticed
          that the Visual Assist plugin did notice that missing
          override. As this uses clang-tidy to determine this, we were
          wondering if these differences we noticed are intentional. If
          so, is their any documentation on these differences?</div>
        <div dir="auto"><br>
        </div>
        <div dir="auto">Tnx</div>
      </div>
      <br>
      <fieldset class="m_4749730040338246124mimeAttachmentHeader"></fieldset>
      <br>
      <pre>_______________________________________________
cfe-dev mailing list
<a class="m_4749730040338246124moz-txt-link-abbreviated" href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>
<a class="m_4749730040338246124moz-txt-link-freetext" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a>
</pre>
    </blockquote>
    <br>
  </div>

_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">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>
</blockquote></div></div>