[mynewt-core] branch master updated: sys/config; allow space in stored config variables.

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

marko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git


The following commit(s) were added to refs/heads/master by this push:
     new 7677674  sys/config; allow space in stored config variables.
     new 0993c9f  Merge pull request #1662 from mkiiskila/config_space_in_value
7677674 is described below

commit 7677674150b0949cac8536bf71ed3fd902947c76
Author: Marko Kiiskila <ma...@apache.org>
AuthorDate: Tue Feb 26 13:41:22 2019 +0200

    sys/config; allow space in stored config variables.
---
 sys/config/src/config_line.c                                | 2 +-
 sys/config/test-nffs/src/testcases/config_test_small_file.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/sys/config/src/config_line.c b/sys/config/src/config_line.c
index 6e6b2ac..5206e75 100644
--- a/sys/config/src/config_line.c
+++ b/sys/config/src/config_line.c
@@ -58,7 +58,7 @@ conf_line_parse(char *buf, char **namep, char **valp)
             }
             break;
         case FIND_VAL_END:
-            if (isspace((unsigned char)*cp)) {
+            if (!isprint((unsigned char)*cp)) {
                 *cp = '\0';
             }
             break;
diff --git a/sys/config/test-nffs/src/testcases/config_test_small_file.c b/sys/config/test-nffs/src/testcases/config_test_small_file.c
index f8c3cf4..d3f14a0 100644
--- a/sys/config/test-nffs/src/testcases/config_test_small_file.c
+++ b/sys/config/test-nffs/src/testcases/config_test_small_file.c
@@ -24,7 +24,7 @@ TEST_CASE(config_test_small_file)
     struct conf_file cf_mfg;
     struct conf_file cf_running;
     const char cf_mfg_test[] = "myfoo/mybar=1";
-    const char cf_running_test[] = " myfoo/mybar = 8 ";
+    const char cf_running_test[] = " myfoo/mybar = 8";
 
     config_wipe_srcs();