forked from OSchip/llvm-project
09464e63d8
Fix for __declspec attributes and const=0 without space
This patch is to address 2 problems I found with Clang-tidy:modernize-use-override.
1: missing spaces on pure function decls.
Orig:
void pure() const=0
Problem:
void pure() constoverride =0
Fixed:
void pure() const override =0
2: This is ms-extension specific, but possibly applies to other attribute types. The override is placed before the attribute which doesn’t work well with declspec as this attribute can be inherited or placed before the method identifier.
Orig:
class __declspec(dllexport) X : public Y
{
void p();
};
Problem:
class override __declspec(dllexport) class X : public Y
{
void p();
};
Fixed:
class __declspec(dllexport) class X : public Y
{
void p() override;
};
Patch by Robert Bolter!
Differential Revision: http://reviews.llvm.org/D18396
llvm-svn: 265298
|
||
|---|---|---|
| .. | ||
| CMakeLists.txt | ||
| DeprecatedHeadersCheck.cpp | ||
| DeprecatedHeadersCheck.h | ||
| LoopConvertCheck.cpp | ||
| LoopConvertCheck.h | ||
| LoopConvertUtils.cpp | ||
| LoopConvertUtils.h | ||
| MakeUniqueCheck.cpp | ||
| MakeUniqueCheck.h | ||
| ModernizeTidyModule.cpp | ||
| PassByValueCheck.cpp | ||
| PassByValueCheck.h | ||
| RawStringLiteralCheck.cpp | ||
| RawStringLiteralCheck.h | ||
| RedundantVoidArgCheck.cpp | ||
| RedundantVoidArgCheck.h | ||
| ReplaceAutoPtrCheck.cpp | ||
| ReplaceAutoPtrCheck.h | ||
| ShrinkToFitCheck.cpp | ||
| ShrinkToFitCheck.h | ||
| UseAutoCheck.cpp | ||
| UseAutoCheck.h | ||
| UseDefaultCheck.cpp | ||
| UseDefaultCheck.h | ||
| UseNullptrCheck.cpp | ||
| UseNullptrCheck.h | ||
| UseOverrideCheck.cpp | ||
| UseOverrideCheck.h | ||