JSON: Switch from StyledWriter to StreamWriterBuilder

diff --git a/httpdclient.c b/httpdclient.c
index e9800710ceec0e12399f23df78b5e2055ba5cbb1..9bfe33f3791d42c2d14230cec3bd2fcbf1ca81ec 100644 (file)
--- a/httpdclient.c
+++ b/httpdclient.c
@@ -6,6 +6,8 @@
 #include <fcntl.h>
 
 #include <memory>
 #include <fcntl.h>
 
 #include <memory>

+#include <string>
+#include <sstream>

 
 #include "httpdclient.h"
 
 
 #include "httpdclient.h"
 


@@ -281,12 +283,14 @@ MHD_Result HTTPDClient::processPOST()
   if (strcmp(url, "/jsonserver")) return sendStockResponse(404);
   if (getVars["req"].empty()) return sendStockResponse(400);
 
   if (strcmp(url, "/jsonserver")) return sendStockResponse(404);
   if (getVars["req"].empty()) return sendStockResponse(400);
 

-  std::string returnData;
+  std::stringstream returnData;

 
 

-  bool success = vdrclient.process(getVars["req"], postFields, returnData);
+  bool success = vdrclient.process(getVars["req"], postFields, &returnData);

   if (!success) return sendStockResponse(500);
 
   if (!success) return sendStockResponse(500);
 

-  struct MHD_Response* response = MHD_create_response_from_buffer(strlen(returnData.c_str()), (void *)returnData.c_str(), MHD_RESPMEM_MUST_COPY);
+  std::string returnDataStr = returnData.str();
+
+  struct MHD_Response* response = MHD_create_response_from_buffer(strlen(returnDataStr.c_str()), (void *)returnDataStr.c_str(), MHD_RESPMEM_MUST_COPY);

   MHD_add_response_header(response, "Content-Type", "application/json");
   MHD_Result ret = MHD_queue_response(mhd_connection, MHD_HTTP_OK, response);
 
   MHD_add_response_header(response, "Content-Type", "application/json");
   MHD_Result ret = MHD_queue_response(mhd_connection, MHD_HTTP_OK, response);
 

diff --git a/vdrclient.c b/vdrclient.c
index 8a1d8b1c1b66d5c1819308c1adc7015d292f6884..669d3b91e7f5c870707057f469b6b6ed94699c54 100644 (file)
--- a/vdrclient.c
+++ b/vdrclient.c
@@ -30,6 +30,11 @@ VDRClient::VDRClient(const std::string& _configDir)
 : configDir(_configDir)
 {
   logger = spd::get("jsonserver_spdlog");
 : configDir(_configDir)
 {
   logger = spd::get("jsonserver_spdlog");

+
+  Json::StreamWriterBuilder jsonWriterBuilder;
+  // jsonWriterBuilder["commentStyle"] = "None";
+  // jsonWriterBuilder["indentation"] = "   ";
+  jsonwriter.reset(jsonWriterBuilder.newStreamWriter());

 }
 
 VDRClient::~VDRClient()
 }
 
 VDRClient::~VDRClient()


@@ -37,7 +42,7 @@ VDRClient::~VDRClient()
   logger->debug("VDRClient destructor");
 }
 
   logger->debug("VDRClient destructor");
 }
 

-bool VDRClient::process(std::string& request, PFMap& postFields, std::string& returnString)
+bool VDRClient::process(std::string& request, PFMap& postFields, std::stringstream* returnStringStream)

 {
   Json::Value returnJSON;
   bool success = false;
 {
   Json::Value returnJSON;
   bool success = false;


@@ -83,9 +88,8 @@ bool VDRClient::process(std::string& request, PFMap& postFields, std::string& re
 
   if (!success) return false;
 
 
   if (!success) return false;
 

-  Json::StyledWriter sw;
-  returnString = sw.write(returnJSON);
-  logger->debug("Done sw write");
+  jsonwriter->write(returnJSON, returnStringStream);
+  logger->debug("Done jsonwriter write");

   return true;
 }
 
   return true;
 }
 

diff --git a/vdrclient.h b/vdrclient.h
index 926f107b28a84466b4a48e43b77359350f471fa7..68e6f1cf871cf14475bb67c9f0b4eba57eec8fad 100644 (file)
--- a/vdrclient.h
+++ b/vdrclient.h
@@ -1,6 +1,8 @@
 #ifndef VDRCLIENT_H
 #define VDRCLIENT_H
 
 #ifndef VDRCLIENT_H
 #define VDRCLIENT_H
 

+#include <sstream>
+

 #include <jsoncpp/json/json.h>
 
 // Log docs: https://github.com/gabime/spdlog
 #include <jsoncpp/json/json.h>
 
 // Log docs: https://github.com/gabime/spdlog


@@ -25,7 +27,7 @@ class VDRClient
     VDRClient(const std::string& configDir);
     ~VDRClient();
 
     VDRClient(const std::string& configDir);
     ~VDRClient();
 

-    bool process(std::string& request, PFMap& postFields, std::string& returnData);
+    bool process(std::string& request, PFMap& postFields, std::stringstream* returnData);

 
   private:
     const std::string& configDir;
 
   private:
     const std::string& configDir;


@@ -86,6 +88,8 @@ class VDRClient
       }
       */
     };
       }
       */
     };

+
+    std::shared_ptr<Json::StreamWriter> jsonwriter;

 };
 
 #endif
 };
 
 #endif