Add namespaces to avoid symbol clashes

Author: sparkprime

Makefile

@@ -27,7 +27,7 @@ OPT ?= -O3
 
 PREFIX ?= /usr/local
 
-CXXFLAGS ?= -g $(OPT) -Wall -Wextra -Woverloaded-virtual -pedantic -std=c++0x -fPIC
+CXXFLAGS ?= -g $(OPT) -Wall -Wextra -Woverloaded-virtual -pedantic -std=c++17 -fPIC
 CXXFLAGS += -Iinclude -Ithird_party/md5 -Ithird_party/json -Ithird_party/rapidyaml/rapidyaml/src/ -Ithird_party/rapidyaml/rapidyaml/ext/c4core/src/
 CFLAGS ?= -g $(OPT) -Wall -Wextra -pedantic -std=c99 -fPIC
 CFLAGS += -Iinclude

core/ast.h

@@ -29,6 +29,8 @@ limitations under the License.
 #include "lexer.h"
 #include "unicode.h"
 
+namespace jsonnet::internal {
+
 enum ASTType {
     AST_APPLY,
     AST_APPLY_BRACE,
@@ -1092,4 +1094,6 @@ auto binary_map = build_binary_map();
 
 }  // namespace
 
+}  // namespace jsonnet::internal
+
 #endif  // JSONNET_AST_H

core/desugarer.cpp

@@ -25,6 +25,8 @@ limitations under the License.
 #include "pass.h"
 #include "string_utils.h"
 
+namespace jsonnet::internal {
+
 static const Fodder EF;  // Empty fodder.
 
 static const LocationRange E;  // Empty.
@@ -1011,3 +1013,5 @@ void jsonnet_desugar(Allocator *alloc, AST *&ast, std::map<std::string, VmExt> *
     Desugarer desugarer(alloc);
     desugarer.desugarFile(ast, tlas);
 }
+
+}  // namespace jsonnet::internal

core/desugarer.h

@@ -23,6 +23,8 @@ limitations under the License.
 #include "ast.h"
 #include "vm.h"
 
+namespace jsonnet::internal {
+
 /** Translate the AST to remove syntax sugar.
  * \param alloc Allocator for making new identifiers / ASTs.
  * \param ast The AST to change.
@@ -37,4 +39,6 @@ void jsonnet_desugar(Allocator *alloc, AST *&ast, std::map<std::string, VmExt> *
  */
 DesugaredObject *makeStdlibAST(Allocator *alloc, std::string filename);
 
+}  // namespace jsonnet::internal
+
 #endif

core/formatter.cpp

@@ -24,6 +24,8 @@ limitations under the License.
 #include "string_utils.h"
 #include "unicode.h"
 
+namespace jsonnet::internal {
+
 static std::string unparse_id(const Identifier *id)
 {
     return encode_utf8(id->name);
@@ -2318,3 +2320,5 @@ std::string jsonnet_fmt(AST *ast, Fodder &final_fodder, const FmtOpts &opts)
     }
     return ss.str();
 }
+
+}  // namespace jsonnet::internal

core/formatter.h

@@ -19,6 +19,8 @@ limitations under the License.
 
 #include "ast.h"
 
+namespace jsonnet::internal {
+
 struct FmtOpts {
     char stringStyle;
     char commentStyle;
@@ -51,4 +53,6 @@ struct FmtOpts {
  */
 std::string jsonnet_fmt(AST *ast, Fodder &final_fodder, const FmtOpts &opts);
 
+}  // namespace jsonnet::internal
+
 #endif  // JSONNET_PARSER_H

core/lexer.cpp

@@ -24,6 +24,8 @@ limitations under the License.
 #include "static_error.h"
 #include "unicode.h"
 
+namespace jsonnet::internal {
+
 static const std::vector<std::string> EMPTY;
 
 /** Is the char whitespace (excluding \n). */
@@ -878,3 +880,5 @@ std::string jsonnet_unlex(const Tokens &tokens)
     }
     return ss.str();
 }
+
+}  // namespace jsonnet::internal

core/lexer.h

@@ -29,6 +29,8 @@ limitations under the License.
 #include "static_error.h"
 #include "unicode.h"
 
+namespace jsonnet::internal {
+
 /** Whitespace and comments.
  *
  * "Fodder" (as in cannon fodder) implies this data is expendable.
@@ -406,4 +408,6 @@ Tokens jsonnet_lex(const std::string &filename, const char *input);
 
 std::string jsonnet_unlex(const Tokens &tokens);
 
+}  // namespace jsonnet::internal
+
 #endif  // JSONNET_LEXER_H

core/lexer_test.cpp

@@ -19,6 +19,7 @@ limitations under the License.
 #include <list>
 #include "gtest/gtest.h"
 
+namespace jsonnet::internal {
 namespace {
 
 void testLex(const char* name, const char* input, const std::list<Token>& tokens,
@@ -342,3 +343,4 @@ TEST(Lexer, TestComments)
 }
 
 }  // namespace
+}  // namespace jsonnet::internal

core/libjsonnet.cpp

@@ -36,6 +36,20 @@ extern "C" {
 #include "static_analysis.h"
 #include "vm.h"
 
+namespace {
+using ::jsonnet::internal::Allocator;
+using ::jsonnet::internal::AST;
+using ::jsonnet::internal::FmtOpts;
+using ::jsonnet::internal::Fodder;
+using ::jsonnet::internal::jsonnet_lex;
+using ::jsonnet::internal::RuntimeError;
+using ::jsonnet::internal::StaticError;
+using ::jsonnet::internal::Tokens;
+using ::jsonnet::internal::VmExt;
+using ::jsonnet::internal::VmNativeCallback;
+using ::jsonnet::internal::VmNativeCallbackMap;
+}  // namespace
+
 static void memory_panic(void)
 {
     fputs("FATAL ERROR: a memory allocation error occurred.\n", stderr);