[PATCH] D18396: Clang-tidy:modernize-use-override. Fix for __declspec attributes and const=0 without spacse
Robert Bolter via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 23 08:02:49 PDT 2016
Rob created this revision.
Rob added reviewers: alexfh, aaron.ballman.
Rob added a subscriber: cfe-commits.
This patch is to address 2 problems I found with Clang-tidy:modernize-use-override.
1: missing spaces on pure function decls.
void pure() const=0
void pure() constoverride =0
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.
class __declspec(dllexport) X : public Y
class override __declspec(dllexport) class X : public Y
class __declspec(dllexport) class X : public Y
void p() override;
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 4220 bytes
Desc: not available
More information about the cfe-commits