Revert "Don't mark all macros with double underscores as reserved."

src/compiler/preprocessor/DirectiveParser.cpp

@@ -118,28 +118,13 @@ void skipUntilEOD(pp::Lexer *lexer, pp::Token *token)
 
 bool isMacroNameReserved(const std::string &name)
 {
-    const char *kPredefinedMacros[] =
-    {
-        "__LINE__",
-        "__FILE__",
-        "__VERSION__",
-        "GL_ES",
-    };
-    const size_t kPredefinedMacrosCount = sizeof(kPredefinedMacros) / sizeof(*kPredefinedMacros);
-
-    for (size_t i = 0; i < kPredefinedMacrosCount; i++)
-    {
-        if (name == kPredefinedMacros[i])
-        {
-            return true;
-        }
-    }
-
     // Names prefixed with "GL_" are reserved.
-    if (name.compare(0, 3, "GL_") == 0)
-    {
+    if (name.substr(0, 3) == "GL_")
+        return true;
+
+    // Names containing two consecutive underscores are reserved.
+    if (name.find("__") != std::string::npos)
         return true;
-    }
 
     return false;
 }

src/tests/preprocessor_tests/define_test.cpp

@@ -65,24 +65,32 @@ TEST_F(DefineTest, RedefinePredefined)
     preprocess(input, expected);
 }
 
-TEST_F(DefineTest, UnderScore1)
+TEST_F(DefineTest, ReservedUnderScore1)
 {
-    const char* input = "#define __foo bar\n";
-    const char* expected = "\n";
+    const char* input = "#define __foo bar\n"
+                        "__foo\n";
+    const char* expected = "\n"
+                           "__foo\n";
 
-    using testing::_;
-    EXPECT_CALL(mDiagnostics, print(_, _, _)).Times(0);
+    EXPECT_CALL(mDiagnostics,
+                print(pp::Diagnostics::PP_MACRO_NAME_RESERVED,
+                      pp::SourceLocation(0, 1),
+                      "__foo"));
 
     preprocess(input, expected);
 }
 
-TEST_F(DefineTest, UnderScore2)
+TEST_F(DefineTest, ReservedUnderScore2)
 {
-    const char* input = "#define foo__bar baz\n";
-    const char* expected = "\n";
+    const char* input = "#define foo__bar baz\n"
+                        "foo__bar\n";
+    const char* expected = "\n"
+                           "foo__bar\n";
 
-    using testing::_;
-    EXPECT_CALL(mDiagnostics, print(_, _, _)).Times(0);
+    EXPECT_CALL(mDiagnostics,
+                print(pp::Diagnostics::PP_MACRO_NAME_RESERVED,
+                      pp::SourceLocation(0, 1),
+                      "foo__bar"));
 
     preprocess(input, expected);
 }