alexfh requested changes to this revision.
alexfh added inline comments.
This revision now requires changes to proceed.
================
Comment at: docs/clang-tidy/checks/bugprone-parent-virtual-call.rst:26
+ int foo() override {... A::foo()...}
+ ^^^^^^^^ warning: qualified name A::foo refers to a
+ member which was overridden in
----------------
alexfh wrote:
> The formatting looks weird. Could you move `warning: .*` to the start of the
> next line and put the whole message on the same line?
Could you address the "and put the whole message on the same line" part as well?
Please also put `// ` before `warning:` and `...`s to make the example look
more like syntactically valid code.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44295
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
