forked from OSchip/llvm-project
68f5e5456f
Summary: The usefulness of **modernize-use-override** can be reduced if you have to live in an environment where you support multiple compilers, some of which sadly are not yet fully C++11 compliant some codebases have to use override as a macro OVERRIDE e.g. ``` // GCC 4.7 supports explicit virtual overrides when C++11 support is enabled. ``` This allows code to be compiled with C++11 compliant compilers and get warnings and errors that clang, MSVC,gcc can give, while still allowing other legacy pre C++11 compilers to compile the code. This can be an important step towards modernizing C++ code whilst living in a legacy codebase. When it comes to clang tidy, the use of the **modernize-use-override** is one of the most useful checks, but the messages reported are inaccurate for that codebase if the standard approach is to use the macros OVERRIDE and/or FINAL. When combined with fix-its that introduce the C++11 override keyword, they become fatal, resulting in the modernize-use-override check being turned off to prevent the introduction of such errors. This revision, allows the possibility for the replacement **override **to be a macro instead, Allowing the clang-tidy check to be run on both pre and post C++11 code, and allowing fix-its to be applied. Reviewers: alexfh, JonasToth, hokein, Eugene.Zelenko, aaron.ballman Reviewed By: alexfh, JonasToth Subscribers: lewmpk, malcolm.parsons, jdoerfert, xazax.hun, cfe-commits, llvm-commits Tags: #clang-tools-extra Differential Revision: https://reviews.llvm.org/D57087 llvm-svn: 355132 |
||
|---|---|---|
| .. | ||
| _static | ||
| _templates | ||
| clang-tidy | ||
| clangd | ||
| CMakeLists.txt | ||
| ModularizeUsage.rst | ||
| README.txt | ||
| ReleaseNotes.rst | ||
| clang-doc.rst | ||
| clang-modernize.rst | ||
| clang-rename.rst | ||
| clang-tidy.rst | ||
| clangd.rst | ||
| conf.py | ||
| cpp11-migrate.rst | ||
| doxygen-mainpage.dox | ||
| doxygen.cfg.in | ||
| include-fixer.rst | ||
| index.rst | ||
| make.bat | ||
| modularize.rst | ||
| pp-trace.rst | ||
README.txt
---------------------------------- Documentation in clang-tools-extra ---------------------------------- To generate documentation in HTML format from files in clang-tools-extra/docs, build the docs-clang-tools-html target. To generate documentation from the source code using Doxygen, build the doxygen-clang-tools target.