[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
Mon Apr 4 05:09:55 PDT 2016
Rob added a comment.
ok, I'm not sure if its worth rolling back the change to modernize-use-override-ms.cpp, see ammended comments above.
Apart from this it should be good to go.
I think I am going to need someone to submit the patch for me, I can't use svn due to 'error 400' so I've been using the git mirror.
Comment at: test/clang-tidy/modernize-use-override-ms.cpp:1
@@ +1,2 @@
+// RUN: %check_clang_tidy %s modernize-use-override %t
> Does this test pass on linux? If no additional compiler arguments needed and it just works, maybe just merge this test code to the main test file?
That's a good question. I have been running it with the additional -fms-extensions argument, but on windows I find it works with or without that option. I havent tested it on linux. I could maybe rummage up a virtual machine and give it a go.
More information about the cfe-commits