Changeset 0c01270 in mainline for uspace/lib/conf/src

Timestamp:

2020-07-05T21:00:33Z (6 years ago)

Author:

Matthieu Riolo <matthieu.riolo@…>

Children:

0f75a63

Parents:

a81a950d

git-author:

Michal Koutný <xm.koutny+hos@…> (2015-05-08 11:02:56)

git-committer:

Matthieu Riolo <matthieu.riolo@…> (2020-07-05 21:00:33)

Message:

libconf: Fix iterator behavior, fix config error notification

Location:

uspace/lib/conf/src

Files:

• configuration.c (modified) (2 diffs)
• ini.c (modified) (1 diff)

uspace/lib/conf/src/configuration.c

@@ -43 +43 @@
                 const char *string = ini_item_iterator_value(it);
                 const size_t lineno = ini_item_iterator_lineno(it);
-                has_error = has_error ||
-                    config_item->parse(string, field_dst, parse, lineno);
+                bool success = config_item->parse(string, field_dst, parse, lineno);
+                has_error = has_error || !success;
         }
 
@@ -104 +104 @@
         }
 
-        return has_error ? EOK : EINVAL;
+        return has_error ? EINVAL : EOK;
 }
 

uspace/lib/conf/src/ini.c

@@ -489 +489 @@
 bool ini_item_iterator_valid(ini_item_iterator_t *iterator)
 {
-        bool empty = (iterator->cur_item != NULL);
-        bool looped = (iterator->cur_item == iterator->first_item);
-        return empty || (looped && iterator->incremented);
+        bool empty = (iterator->cur_item == NULL);
+        bool maybe_looped = (iterator->cur_item == iterator->first_item);
+        return !(empty || (maybe_looped && iterator->incremented));
 }