[clangd][NFC] Get rid of raw string literals in macros to make stage1 compiler happy
This commit is contained in:
Kadir Cetinkaya
parent
417dd67825
commit
43e931cb5f
|
|
@ -36,6 +36,7 @@
|
|||
#include "gtest/gtest.h"
|
||||
#include <cassert>
|
||||
#include <string>
|
||||
#include <utility>
|
||||
#include <vector>
|
||||
|
||||
using ::testing::AllOf;
|
||||
|
|
@ -1054,7 +1055,7 @@ TEST_F(DefineInlineTest, UsingShadowDecls) {
|
|||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformNestedNamespaces) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
namespace a {
|
||||
void bar();
|
||||
namespace b {
|
||||
|
|
@ -1081,7 +1082,8 @@ TEST_F(DefineInlineTest, TransformNestedNamespaces) {
|
|||
c::aux();
|
||||
b::c::aux();
|
||||
a::b::c::aux();
|
||||
})cpp"), R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace a {
|
||||
void bar();
|
||||
namespace b {
|
||||
|
|
@ -1108,11 +1110,12 @@ TEST_F(DefineInlineTest, TransformNestedNamespaces) {
|
|||
using namespace a;
|
||||
using namespace b;
|
||||
using namespace c;
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformUsings) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
namespace a { namespace b { namespace c { void aux(); } } }
|
||||
|
||||
void foo();
|
||||
|
|
@ -1122,8 +1125,8 @@ TEST_F(DefineInlineTest, TransformUsings) {
|
|||
using namespace c;
|
||||
using c::aux;
|
||||
namespace d = c;
|
||||
})cpp"),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace a { namespace b { namespace c { void aux(); } } }
|
||||
|
||||
void foo(){
|
||||
|
|
@ -1133,11 +1136,12 @@ TEST_F(DefineInlineTest, TransformUsings) {
|
|||
using a::b::c::aux;
|
||||
namespace d = a::b::c;
|
||||
}
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformDecls) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
void foo();
|
||||
void f^oo() {
|
||||
class Foo {
|
||||
|
|
@ -1153,8 +1157,8 @@ TEST_F(DefineInlineTest, TransformDecls) {
|
|||
|
||||
enum class EnClass { Zero, One };
|
||||
EnClass y = EnClass::Zero;
|
||||
})cpp"),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
void foo(){
|
||||
class Foo {
|
||||
public:
|
||||
|
|
@ -1170,11 +1174,12 @@ TEST_F(DefineInlineTest, TransformDecls) {
|
|||
enum class EnClass { Zero, One };
|
||||
EnClass y = EnClass::Zero;
|
||||
}
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformTemplDecls) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {
|
||||
public:
|
||||
|
|
@ -1190,8 +1195,8 @@ TEST_F(DefineInlineTest, TransformTemplDecls) {
|
|||
void f^oo() {
|
||||
bar<Bar<int>>.bar();
|
||||
aux<Bar<int>>();
|
||||
})cpp"),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {
|
||||
public:
|
||||
|
|
@ -1207,26 +1212,28 @@ TEST_F(DefineInlineTest, TransformTemplDecls) {
|
|||
}
|
||||
|
||||
using namespace a;
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformMembers) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
class Foo {
|
||||
void foo();
|
||||
};
|
||||
|
||||
void Foo::f^oo() {
|
||||
return;
|
||||
})cpp"),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
class Foo {
|
||||
void foo(){
|
||||
return;
|
||||
}
|
||||
};
|
||||
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
|
||||
ExtraFiles["a.h"] = R"cpp(
|
||||
class Foo {
|
||||
|
|
@ -1234,27 +1241,28 @@ TEST_F(DefineInlineTest, TransformMembers) {
|
|||
};)cpp";
|
||||
|
||||
llvm::StringMap<std::string> EditedFiles;
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
Test = R"cpp(
|
||||
#include "a.h"
|
||||
void Foo::f^oo() {
|
||||
return;
|
||||
})cpp",
|
||||
&EditedFiles),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
Expected = R"cpp(
|
||||
#include "a.h"
|
||||
)cpp");
|
||||
EXPECT_THAT(EditedFiles,
|
||||
ElementsAre(FileWithContents(testPath("a.h"),
|
||||
R"cpp(
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test, &EditedFiles), Expected);
|
||||
|
||||
Expected = R"cpp(
|
||||
class Foo {
|
||||
void foo(){
|
||||
return;
|
||||
}
|
||||
};)cpp")));
|
||||
};)cpp";
|
||||
EXPECT_THAT(EditedFiles,
|
||||
ElementsAre(FileWithContents(testPath("a.h"), Expected)));
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformDependentTypes) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {};
|
||||
}
|
||||
|
|
@ -1267,8 +1275,8 @@ TEST_F(DefineInlineTest, TransformDependentTypes) {
|
|||
void f^oo() {
|
||||
Bar<T> B;
|
||||
Bar<Bar<T>> q;
|
||||
})cpp"),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {};
|
||||
}
|
||||
|
|
@ -1280,12 +1288,14 @@ TEST_F(DefineInlineTest, TransformDependentTypes) {
|
|||
}
|
||||
|
||||
using namespace a;
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformFunctionTempls) {
|
||||
// Check we select correct specialization decl.
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
std::pair<llvm::StringRef, llvm::StringRef> Cases[] = {
|
||||
{R"cpp(
|
||||
template <typename T>
|
||||
void foo(T p);
|
||||
|
||||
|
|
@ -1298,7 +1308,7 @@ TEST_F(DefineInlineTest, TransformFunctionTempls) {
|
|||
template <>
|
||||
void fo^o<int>(int p) {
|
||||
return;
|
||||
})cpp"),
|
||||
})cpp",
|
||||
R"cpp(
|
||||
template <typename T>
|
||||
void foo(T p);
|
||||
|
|
@ -1311,10 +1321,9 @@ TEST_F(DefineInlineTest, TransformFunctionTempls) {
|
|||
template <>
|
||||
void foo<char>(char p);
|
||||
|
||||
)cpp");
|
||||
|
||||
// Make sure we are not selecting the first specialization all the time.
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
)cpp"},
|
||||
{// Make sure we are not selecting the first specialization all the time.
|
||||
R"cpp(
|
||||
template <typename T>
|
||||
void foo(T p);
|
||||
|
||||
|
|
@ -1327,7 +1336,7 @@ TEST_F(DefineInlineTest, TransformFunctionTempls) {
|
|||
template <>
|
||||
void fo^o<char>(char p) {
|
||||
return;
|
||||
})cpp"),
|
||||
})cpp",
|
||||
R"cpp(
|
||||
template <typename T>
|
||||
void foo(T p);
|
||||
|
|
@ -1340,9 +1349,8 @@ TEST_F(DefineInlineTest, TransformFunctionTempls) {
|
|||
return;
|
||||
}
|
||||
|
||||
)cpp");
|
||||
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
)cpp"},
|
||||
{R"cpp(
|
||||
template <typename T>
|
||||
void foo(T p);
|
||||
|
||||
|
|
@ -1352,7 +1360,7 @@ TEST_F(DefineInlineTest, TransformFunctionTempls) {
|
|||
template <typename T>
|
||||
void fo^o(T p) {
|
||||
return;
|
||||
})cpp"),
|
||||
})cpp",
|
||||
R"cpp(
|
||||
template <typename T>
|
||||
void foo(T p){
|
||||
|
|
@ -1362,11 +1370,14 @@ TEST_F(DefineInlineTest, TransformFunctionTempls) {
|
|||
template <>
|
||||
void foo<int>(int p);
|
||||
|
||||
)cpp");
|
||||
)cpp"},
|
||||
};
|
||||
for(const auto &Case : Cases)
|
||||
EXPECT_EQ(apply(Case.first), Case.second) << Case.first;
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformTypeLocs) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {
|
||||
public:
|
||||
|
|
@ -1382,8 +1393,8 @@ TEST_F(DefineInlineTest, TransformTypeLocs) {
|
|||
Bar<int> B;
|
||||
Foo foo;
|
||||
a::Bar<Bar<int>>::Baz<Bar<int>> q;
|
||||
})cpp"),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {
|
||||
public:
|
||||
|
|
@ -1399,11 +1410,12 @@ TEST_F(DefineInlineTest, TransformTypeLocs) {
|
|||
}
|
||||
|
||||
using namespace a;
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformDeclRefs) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test =R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {
|
||||
public:
|
||||
|
|
@ -1428,8 +1440,8 @@ TEST_F(DefineInlineTest, TransformDeclRefs) {
|
|||
Bar<int>::y = 3;
|
||||
bar();
|
||||
a::test();
|
||||
})cpp"),
|
||||
R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace a {
|
||||
template <typename T> class Bar {
|
||||
public:
|
||||
|
|
@ -1454,55 +1466,64 @@ TEST_F(DefineInlineTest, TransformDeclRefs) {
|
|||
a::test();
|
||||
}
|
||||
using namespace a;
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, StaticMembers) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
namespace ns { class X { static void foo(); void bar(); }; }
|
||||
void ns::X::b^ar() {
|
||||
foo();
|
||||
})cpp"), R"cpp(
|
||||
})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace ns { class X { static void foo(); void bar(){
|
||||
foo();
|
||||
} }; }
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TransformInlineNamespaces) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
auto Test = R"cpp(
|
||||
namespace a { inline namespace b { namespace { struct Foo{}; } } }
|
||||
void foo();
|
||||
|
||||
using namespace a;
|
||||
void ^foo() {Foo foo;})cpp"), R"cpp(
|
||||
void ^foo() {Foo foo;})cpp";
|
||||
auto Expected = R"cpp(
|
||||
namespace a { inline namespace b { namespace { struct Foo{}; } } }
|
||||
void foo(){a::Foo foo;}
|
||||
|
||||
using namespace a;
|
||||
)cpp");
|
||||
)cpp";
|
||||
EXPECT_EQ(apply(Test), Expected);
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, TokensBeforeSemicolon) {
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
std::pair<llvm::StringRef, llvm::StringRef> Cases[] = {
|
||||
{R"cpp(
|
||||
void foo() /*Comment -_-*/ /*Com 2*/ ;
|
||||
void fo^o() { return ; })cpp"),
|
||||
void fo^o() { return ; })cpp",
|
||||
R"cpp(
|
||||
void foo() /*Comment -_-*/ /*Com 2*/ { return ; }
|
||||
)cpp");
|
||||
)cpp"},
|
||||
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
{R"cpp(
|
||||
void foo();
|
||||
void fo^o() { return ; })cpp"),
|
||||
void fo^o() { return ; })cpp",
|
||||
R"cpp(
|
||||
void foo(){ return ; }
|
||||
)cpp");
|
||||
)cpp"},
|
||||
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
{R"cpp(
|
||||
#define SEMI ;
|
||||
void foo() SEMI
|
||||
void fo^o() { return ; })cpp"),
|
||||
"fail: Couldn't find semicolon for target declaration.");
|
||||
void fo^o() { return ; })cpp",
|
||||
"fail: Couldn't find semicolon for target declaration."},
|
||||
};
|
||||
for(const auto& Case: Cases)
|
||||
EXPECT_EQ(apply(Case.first), Case.second) << Case.first;
|
||||
}
|
||||
|
||||
TEST_F(DefineInlineTest, HandleMacros) {
|
||||
|
|
@ -1516,47 +1537,51 @@ TEST_F(DefineInlineTest, HandleMacros) {
|
|||
void foo();
|
||||
[[BODY]])cpp");
|
||||
|
||||
std::pair<llvm::StringRef, llvm::StringRef> Cases[] = {
|
||||
// We don't qualify declarations coming from macros.
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
{R"cpp(
|
||||
#define BODY Foo
|
||||
namespace a { class Foo{}; }
|
||||
void foo();
|
||||
using namespace a;
|
||||
void f^oo(){BODY})cpp"),
|
||||
void f^oo(){BODY})cpp",
|
||||
R"cpp(
|
||||
#define BODY Foo
|
||||
namespace a { class Foo{}; }
|
||||
void foo(){BODY}
|
||||
using namespace a;
|
||||
)cpp");
|
||||
)cpp"},
|
||||
|
||||
// Macro is not visible at declaration location, but we proceed.
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
{R"cpp(
|
||||
void foo();
|
||||
#define BODY return;
|
||||
void f^oo(){BODY})cpp"),
|
||||
void f^oo(){BODY})cpp",
|
||||
R"cpp(
|
||||
void foo(){BODY}
|
||||
#define BODY return;
|
||||
)cpp");
|
||||
)cpp"},
|
||||
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
{R"cpp(
|
||||
#define TARGET void foo()
|
||||
TARGET;
|
||||
void f^oo(){ return; })cpp"),
|
||||
void f^oo(){ return; })cpp",
|
||||
R"cpp(
|
||||
#define TARGET void foo()
|
||||
TARGET{ return; }
|
||||
)cpp");
|
||||
)cpp"},
|
||||
|
||||
EXPECT_EQ(apply(R"cpp(
|
||||
{R"cpp(
|
||||
#define TARGET foo
|
||||
void TARGET();
|
||||
void f^oo(){ return; })cpp"),
|
||||
void f^oo(){ return; })cpp",
|
||||
R"cpp(
|
||||
#define TARGET foo
|
||||
void TARGET(){ return; }
|
||||
)cpp");
|
||||
)cpp"},
|
||||
};
|
||||
for(const auto& Case: Cases)
|
||||
EXPECT_EQ(apply(Case.first), Case.second) << Case.first;
|
||||
}
|
||||
|
||||
} // namespace
|
||||
|
|
|
|||
Loading…
Reference in New Issue