[Libreoffice-commits] online.git: Branch 'distro/collabora/collabora-online-4' - common/Util.cpp

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Apr 15 02:23:47 UTC 2019


 common/Util.cpp |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 3e8d395618c6dfbf952fd632ea26c8ffafd39ee8
Author:     Ashod Nakashian <ashod.nakashian at collabora.co.uk>
AuthorDate: Sat Mar 9 21:21:48 2019 -0500
Commit:     Ashod Nakashian <ashnakash at gmail.com>
CommitDate: Mon Apr 15 04:23:30 2019 +0200

    wsd: parse empty json string as empty map
    
    Change-Id: I11335e82e42e9f896ae33c2e57cb9d79b642171e
    Reviewed-on: https://gerrit.libreoffice.org/69633
    Reviewed-by: Ashod Nakashian <ashnakash at gmail.com>
    Tested-by: Ashod Nakashian <ashnakash at gmail.com>

diff --git a/common/Util.cpp b/common/Util.cpp
index b0b870605..44a39fea8 100644
--- a/common/Util.cpp
+++ b/common/Util.cpp
@@ -571,6 +571,10 @@ namespace Util
 
     std::map<std::string, std::string> JsonToMap(const std::string& jsonString)
     {
+        std::map<std::string, std::string> map;
+        if (jsonString.empty())
+            return map;
+
         Poco::JSON::Parser parser;
         const Poco::Dynamic::Var result = parser.parse(jsonString);
         const auto& json = result.extract<Poco::JSON::Object::Ptr>();
@@ -578,7 +582,6 @@ namespace Util
         std::vector<std::string> names;
         json->getNames(names);
 
-        std::map<std::string, std::string> map;
         for (const auto& name : names)
         {
             map[name] = json->get(name).toString();


More information about the Libreoffice-commits mailing list