Support "=" in sub-config file path.

kxs-bpei · wilx · commit 28a6e8a3a031 · 2019-10-21T19:55:53.000+02:00
If there is a "=" in the path of sub-config file, and we include it in
the main config file like:

```
include D:/path/to/sub=config.ini
```

it will not work.

Because we parse line with "=" in it as a key/value pair.

This PR is going to fix this issue and support "=" in the "include" directive.

Signed-off-by: Vaclav Haisman &lt;vhaisman@gmail.com&gt;
diff --git a/src/property.cxx b/src/property.cxx
@@ -247,17 +247,8 @@ Properties::init(tistream& input)
             // Remove trailing 'Windows' \r.
             buffer.resize (buffLen - 1);
 
-        tstring::size_type const idx = buffer.find(LOG4CPLUS_TEXT ('='));
-        if (idx != tstring::npos)
-        {
-            tstring key = buffer.substr(0, idx);
-            tstring value = buffer.substr(idx + 1);
-            trim_trailing_ws (key);
-            trim_ws (value);
-            setProperty(key, value);
-        }
-        else if (buffer.compare (0, 7, LOG4CPLUS_TEXT ("include")) == 0
-            && buffer.size () >= 7 + 1 + 1
+        if (buffer.size () >= 7 + 1 + 1
+            && buffer.compare (0, 7, LOG4CPLUS_TEXT ("include")) == 0
             && is_space (buffer[7]))
         {
             tstring included (buffer, 8) ;
@@ -274,6 +265,18 @@ Properties::init(tistream& input)
 
             init (file);
         }
+        else
+        {
+            tstring::size_type const idx = buffer.find(LOG4CPLUS_TEXT ('='));
+            if (idx != tstring::npos)
+            {
+                tstring key = buffer.substr(0, idx);
+                tstring value = buffer.substr(idx + 1);
+                trim_trailing_ws (key);
+                trim_ws (value);
+                setProperty(key, value);
+            }
+        }
     }
 }
 
