Replace cJSON value by our own. (#3837)

Author: Jarod42

CodeFormatter/CodeFormatterManager.cpp

@@ -86,7 +86,7 @@ void CodeFormatterManager::Save()
 {
     wxFileName config_file{ clStandardPaths::Get().GetUserDataDir(), "code-formatters.json" };
     config_file.AppendDir("config");
-    JSON root{ cJSON_Array };
+    JSON root{JsonType::Array};
     auto arr = root.toElement();
     for (auto fmtr : m_formatters) {
         arr.arrayAppend(fmtr->ToJSON());

CodeLite/JSON.cpp

@@ -29,9 +29,6 @@
 #include "clFontHelper.h"
 #include "fileutils.h"
 
-#include <stdlib.h>
-#include <wx/dynarray.h>
-#include <wx/ffile.h>
 #include <wx/filename.h>
 
 JSON::JSON(const wxString& text)
@@ -50,12 +47,12 @@ JSON::JSON(JSONItem item)
 {
 }
 
-JSON::JSON(int type)
+JSON::JSON(JsonType type)
     : m_json(NULL)
 {
-    if (type == cJSON_Array)
+    if (type == JsonType::Array)
         m_json = cJSON_CreateArray();
-    else if (type == cJSON_NULL)
+    else if (type == JsonType::Null)
         m_json = cJSON_CreateNull();
     else
         m_json = cJSON_CreateObject();

CodeLite/JSON.h

@@ -258,6 +258,12 @@ class WXDLLIMPEXP_CL JSONItem
 
 //////////////////////////////////////////////////////////////////////////
 //////////////////////////////////////////////////////////////////////////
+enum class JsonType
+{
+    Array,
+    Null,
+    Object,
+};
 
 class WXDLLIMPEXP_CL JSON
 {
@@ -266,11 +272,12 @@ class WXDLLIMPEXP_CL JSON
     wxString _errorString;
 
 public:
-    JSON(int type);
-    JSON(const wxString& text);
-    JSON(const wxFileName& filename);
-    JSON(JSONItem item);
-    JSON(cJSON* json);
+    explicit JSON(JsonType type);
+    explicit JSON(const wxString& text);
+    explicit JSON(const wxFileName& filename);
+    explicit JSON(JSONItem item);
+    explicit JSON(cJSON* json);
+
     virtual ~JSON();
 
     void save(const wxFileName& fn) const;

CodeLite/LSP/InitializeRequest.cpp

@@ -17,7 +17,7 @@ JSONItem LSP::InitializeRequest::ToJSON(const wxString& name) const
     json.append(params);
     params.addProperty("processId", GetProcessId());
     if (GetRootUri().IsEmpty()) {
-        JSON nullItem(cJSON_NULL);
+        JSON nullItem(JsonType::Null);
         JSONItem nullObj = nullItem.toElement();
         params.append(nullObj);
         (void)nullItem.release(); // don't delete it on destruction, it is now owned by 'params'

CodeLite/cl_config.cpp

@@ -70,7 +70,7 @@ clConfig::clConfig(const wxString& filename)
         if (!m_filename.DirExists()) {
             m_filename.Mkdir(wxS_DIR_DEFAULT, wxPATH_MKDIR_FULL);
         }
-        m_root = std::make_unique<JSON>(cJSON_Object);
+        m_root = std::make_unique<JSON>(JsonType::Object);
     }
 
     // Load the "Recent Items" cache

DebugAdapterClient/clDapSettingsStore.cpp

@@ -60,7 +60,7 @@ void clDapSettingsStore::Save(const wxFileName& path)
     if(!path.IsOk()) {
         return;
     }
-    JSON root(cJSON_Array);
+    JSON root(JsonType::Array);
     for(const auto& vt : m_entries) {
         root.toElement().arrayAppend(vt.second.To());
     }

LiteEditor/app.cpp

@@ -868,7 +868,7 @@ bool CodeLiteApp::IsSingleInstance(const wxCmdLineParser& m_parser)
                 bool dummy;
                 client.ConnectRemote("127.0.0.1", SINGLE_INSTANCE_PORT, dummy);
 
-                JSON json(cJSON_Object);
+                JSON json(JsonType::Object);
                 json.toElement().addProperty("args", files);
 
                 long lineNumber(0);

Plugin/ColoursAndFontsManager.cpp

@@ -382,7 +382,7 @@ LexerConf::Ptr_t ColoursAndFontsManager::GetLexer(const wxString& lexerName, con
 void ColoursAndFontsManager::Save(const wxFileName& lexer_json)
 {
     bool for_export = lexer_json.IsOk();
-    JSON root(cJSON_Array);
+    JSON root(JsonType::Array);
     JSONItem element = root.toElement();
     for (const auto& [_, lexers] : m_lexersMap) {
         for (const auto& lexer : lexers) {
@@ -578,7 +578,7 @@ wxFileName ColoursAndFontsManager::GetConfigFile() const
 void ColoursAndFontsManager::SaveGlobalSettings(bool notify)
 {
     // save the global settings
-    JSON root(cJSON_Object);
+    JSON root(JsonType::Object);
     root.toElement().addProperty("m_globalTheme", m_globalTheme);
     wxFileName fnSettings = GetConfigFile();
     root.save(fnSettings.GetFullPath());
@@ -1162,7 +1162,7 @@ bool ColoursAndFontsManager::ExportThemesToFile(const wxFileName& outputFile, co
         M.insert(names.Item(i).Lower());
     }
 
-    JSON root(cJSON_Array);
+    JSON root(JsonType::Array);
     JSONItem arr = root.toElement();
     std::vector<LexerConf::Ptr_t> Lexers;
     for (const auto& lexer : m_allLexers) {

Plugin/FileSystemWorkspace/clFileSystemWorkspaceConfig.cpp

@@ -372,8 +372,8 @@ bool clFileSystemWorkspaceSettings::Save(const wxFileName& filename, const wxFil
         localWorkspace.AppendDir(".codelite");
     }
     localWorkspace.Mkdir(wxS_DIR_DEFAULT, wxPATH_MKDIR_FULL);
-    JSON root_local(cJSON_Object);
-    JSON root_shared(cJSON_Object);
+    JSON root_local(JsonType::Object);
+    JSON root_shared(JsonType::Object);
 
     JSONItem itemLocal = root_local.toElement();
     JSONItem itemShared = root_shared.toElement();

Plugin/Keyboard/clKeyboardBindingConfig.cpp

@@ -57,7 +57,7 @@ clKeyboardBindingConfig& clKeyboardBindingConfig::Load()
 
 clKeyboardBindingConfig& clKeyboardBindingConfig::Save()
 {
-    JSON root(cJSON_Object);
+    JSON root(JsonType::Object);
     JSONItem mainObj = root.toElement();
 
     // set the version