[cfe-dev] Tidy vs Warnings: missing override

[Jonas Toth via cfe-dev]

That would be worth a bug-report :)


Am 10.08.2018 um 15:03 schrieb Matthieu Brucher:
> Hi,
>
> I have noticed the missing override warning when using virtual diamond
> inheritance. In that case, there was no warning generated.
>
> Regards,
>
> Matthieu
>
> Le ven. 10 août 2018 à 10:23, Jonas Toth via cfe-dev
> <cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>> a écrit :
>
>     Hi,
>
>     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.
>
>     Could this be the case?
>
>     Best, Jonas
>
>
>     Am 09.08.2018 um 19:26 schrieb JVApen via cfe-dev:
>>     Hello all,
>>
>>     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.
>>
>>     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?
>>
>>     Tnx
>>
>>
>>     _______________________________________________
>>     cfe-dev mailing list
>>     cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>
>>     http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>
>     _______________________________________________
>     cfe-dev mailing list
>     cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>
>     http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>
>
>
> -- 
> Quantitative analyst, Ph.D.
> Blog: http://blog.audio-tk.com/
> LinkedIn: http://www.linkedin.com/in/matthieubrucher

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20180810/7a1e2ef1/attachment.html>