diff options
| author | Pekka Paalanen <ppaalanen@gmail.com> | 2011-12-08 09:24:24 +0200 |
|---|---|---|
| committer | Pekka Paalanen <ppaalanen@gmail.com> | 2011-12-08 10:25:54 +0200 |
| commit | 28a20707a291854be335b800c741bdaf2c19a74b (patch) | |
| tree | 6084374820243c427b2ddabc2e954fc937ef91e9 /shared | |
| parent | c1765c67b394ec82c5d576140e0e297088deaabb (diff) | |
configparser: rename BOOL, document types
Rename CONFIG_KEY_BOOL to CONFIG_KEY_BOOLEAN, just like
CONFIG_KEY_INTEGER is not CONFIG_KEY_INT, for consistency.
Document the types expected for the void* and name the enum in the
header, so it is clear what config_key::type means.
Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>
Diffstat (limited to 'shared')
| -rw-r--r-- | shared/configparser.c | 2 | ||||
| -rw-r--r-- | shared/configparser.h | 10 |
2 files changed, 6 insertions, 6 deletions
diff --git a/shared/configparser.c b/shared/configparser.c index c48fe4c..3deb89d 100644 --- a/shared/configparser.c +++ b/shared/configparser.c @@ -53,7 +53,7 @@ handle_key(const struct config_key *key, const char *value) *(char **)key->data = s; return 0; - case CONFIG_KEY_BOOL: + case CONFIG_KEY_BOOLEAN: if (strcmp(value, "false\n") == 0) *(int *)key->data = 0; else if (strcmp(value, "true\n") == 0) diff --git a/shared/configparser.h b/shared/configparser.h index 93b335c..34f5c31 100644 --- a/shared/configparser.h +++ b/shared/configparser.h @@ -23,15 +23,15 @@ #ifndef CONFIGPARSER_H #define CONFIGPARSER_H -enum { - CONFIG_KEY_INTEGER, - CONFIG_KEY_STRING, - CONFIG_KEY_BOOL +enum config_key_type { + CONFIG_KEY_INTEGER, /* typeof data = int */ + CONFIG_KEY_STRING, /* typeof data = char* */ + CONFIG_KEY_BOOLEAN /* typeof data = int */ }; struct config_key { const char *name; - int type; + enum config_key_type type; void *data; }; |