p
/
xK
1
1
Fork 0
Code Issues 1 Pull Requests Releases 10 Wiki Activity

mv 'struct config_item'{_,} 

Finally we can get rid of the trailing underscore.
This commit is contained in:
Přemysl Eric Janouch 2015-08-17 00:08:19 +02:00
parent abd892cbd7
commit 1613e75a48
2 changed files with 98 additions and 98 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

106
common.c
View File

@ -1014,7 +1014,7 @@ enum config_item_type
CONFIG_ITEM_STRING_ARRAY ///< Comma-separated list of strings CONFIG_ITEM_STRING_ARRAY ///< Comma-separated list of strings
}; };
struct config_item_ struct config_item
{ {
enum config_item_type type; ///< Type of the item enum config_item_type type; ///< Type of the item
union union
@ -1040,10 +1040,10 @@ struct config_schema
/// Check if the new value can be accepted. /// Check if the new value can be accepted.
/// In addition to this, "type" and having a default is considered. /// In addition to this, "type" and having a default is considered.
bool (*validate) (const struct config_item_ *, struct error **e); bool (*validate) (const struct config_item *, struct error **e);
/// The value has changed /// The value has changed
void (*on_change) (struct config_item_ *); void (*on_change) (struct config_item *);
}; };
// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
@ -1073,7 +1073,7 @@ config_item_type_is_string (enum config_item_type type)
} }
static void static void
config_item_free (struct config_item_ *self) config_item_free (struct config_item *self)
{ {
switch (self->type) switch (self->type)
{ {
@ -1089,7 +1089,7 @@ config_item_free (struct config_item_ *self)
} }
static void static void
config_item_destroy (struct config_item_ *self) config_item_destroy (struct config_item *self)
{ {
config_item_free (self); config_item_free (self);
free (self); free (self);
@ -1098,7 +1098,7 @@ config_item_destroy (struct config_item_ *self)
/// Doesn't do any validations or handle schemas, just moves source data /// Doesn't do any validations or handle schemas, just moves source data
/// to the target item and destroys the source item /// to the target item and destroys the source item
static void static void
config_item_move (struct config_item_ *self, struct config_item_ *source) config_item_move (struct config_item *self, struct config_item *source)
{ {
// Not quite sure how to handle that // Not quite sure how to handle that
hard_assert (!source->schema); hard_assert (!source->schema);
@ -1109,40 +1109,40 @@ config_item_move (struct config_item_ *self, struct config_item_ *source)
free (source); free (source);
} }
static struct config_item_ * static struct config_item *
config_item_new (enum config_item_type type) config_item_new (enum config_item_type type)
{ {
struct config_item_ *self = xcalloc (1, sizeof *self); struct config_item *self = xcalloc (1, sizeof *self);
self->type = type; self->type = type;
return self; return self;
} }
static struct config_item_ * static struct config_item *
config_item_null (void) config_item_null (void)
{ {
return config_item_new (CONFIG_ITEM_NULL); return config_item_new (CONFIG_ITEM_NULL);
} }
static struct config_item_ * static struct config_item *
config_item_boolean (bool b) config_item_boolean (bool b)
{ {
struct config_item_ *self = config_item_new (CONFIG_ITEM_BOOLEAN); struct config_item *self = config_item_new (CONFIG_ITEM_BOOLEAN);
self->value.boolean = b; self->value.boolean = b;
return self; return self;
} }
static struct config_item_ * static struct config_item *
config_item_integer (int64_t i) config_item_integer (int64_t i)
{ {
struct config_item_ *self = config_item_new (CONFIG_ITEM_INTEGER); struct config_item *self = config_item_new (CONFIG_ITEM_INTEGER);
self->value.integer = i; self->value.integer = i;
return self; return self;
} }
static struct config_item_ * static struct config_item *
config_item_string (const struct str *s) config_item_string (const struct str *s)
{ {
struct config_item_ *self = config_item_new (CONFIG_ITEM_STRING); struct config_item *self = config_item_new (CONFIG_ITEM_STRING);
str_init (&self->value.string); str_init (&self->value.string);
hard_assert (utf8_validate hard_assert (utf8_validate
(self->value.string.str, self->value.string.len)); (self->value.string.str, self->value.string.len));
@ -1150,29 +1150,29 @@ config_item_string (const struct str *s)
return self; return self;
} }
static struct config_item_ * static struct config_item *
config_item_string_from_cstr (const char *s) config_item_string_from_cstr (const char *s)
{ {
struct str tmp; struct str tmp;
str_init (&tmp); str_init (&tmp);
str_append (&tmp, s); str_append (&tmp, s);
struct config_item_ *self = config_item_string (&tmp); struct config_item *self = config_item_string (&tmp);
str_free (&tmp); str_free (&tmp);
return self; return self;
} }
static struct config_item_ * static struct config_item *
config_item_string_array (const struct str *s) config_item_string_array (const struct str *s)
{ {
struct config_item_ *self = config_item_string (s); struct config_item *self = config_item_string (s);
self->type = CONFIG_ITEM_STRING_ARRAY; self->type = CONFIG_ITEM_STRING_ARRAY;
return self; return self;
} }
static struct config_item_ * static struct config_item *
config_item_object (void) config_item_object (void)
{ {
struct config_item_ *self = config_item_new (CONFIG_ITEM_OBJECT); struct config_item *self = config_item_new (CONFIG_ITEM_OBJECT);
str_map_init (&self->value.object); str_map_init (&self->value.object);
self->value.object.free = (void (*)(void *)) config_item_destroy; self->value.object.free = (void (*)(void *)) config_item_destroy;
return self; return self;
@ -1192,7 +1192,7 @@ config_schema_accepts_type
} }
static bool static bool
config_item_validate_by_schema (struct config_item_ *self, config_item_validate_by_schema (struct config_item *self,
struct config_schema *schema, struct error **e) struct config_schema *schema, struct error **e)
{ {
struct error *error = NULL; struct error *error = NULL;
@ -1211,7 +1211,7 @@ config_item_validate_by_schema (struct config_item_ *self,
} }
static bool static bool
config_item_set_from (struct config_item_ *self, struct config_item_ *source, config_item_set_from (struct config_item *self, struct config_item *source,
struct error **e) struct error **e)
{ {
struct config_schema *schema = self->schema; struct config_schema *schema = self->schema;
@ -1238,8 +1238,8 @@ config_item_set_from (struct config_item_ *self, struct config_item_ *source,
return true; return true;
} }
static struct config_item_ * static struct config_item *
config_item_get (struct config_item_ *self, const char *path, struct error **e) config_item_get (struct config_item *self, const char *path, struct error **e)
{ {
hard_assert (self->type == CONFIG_ITEM_OBJECT); hard_assert (self->type == CONFIG_ITEM_OBJECT);
@ -1247,7 +1247,7 @@ config_item_get (struct config_item_ *self, const char *path, struct error **e)
str_vector_init (&v); str_vector_init (&v);
split_str (path, '.', &v); split_str (path, '.', &v);
struct config_item_ *result = NULL; struct config_item *result = NULL;
size_t i = 0; size_t i = 0;
while (true) while (true)
{ {
@ -1277,7 +1277,7 @@ struct config_writer
}; };
static void config_item_write_object_innards static void config_item_write_object_innards
(struct config_writer *self, struct config_item_ *object); (struct config_writer *self, struct config_item *object);
static void static void
config_item_write_string (struct str *output, const struct str *s) config_item_write_string (struct str *output, const struct str *s)
@ -1299,7 +1299,7 @@ config_item_write_string (struct str *output, const struct str *s)
static void static void
config_item_write_object config_item_write_object
(struct config_writer *self, struct config_item_ *value) (struct config_writer *self, struct config_item *value)
{ {
char indent[self->indent + 1]; char indent[self->indent + 1];
memset (indent, '\t', self->indent); memset (indent, '\t', self->indent);
@ -1318,7 +1318,7 @@ config_item_write_object
} }
static void static void
config_item_write_value (struct config_writer *self, struct config_item_ *value) config_item_write_value (struct config_writer *self, struct config_item *value)
{ {
switch (value->type) switch (value->type)
{ {
@ -1345,7 +1345,7 @@ config_item_write_value (struct config_writer *self, struct config_item_ *value)
static void static void
config_item_write_kv_pair (struct config_writer *self, config_item_write_kv_pair (struct config_writer *self,
const char *key, struct config_item_ *value) const char *key, struct config_item *value)
{ {
char indent[self->indent + 1]; char indent[self->indent + 1];
memset (indent, '\t', self->indent); memset (indent, '\t', self->indent);
@ -1362,20 +1362,20 @@ config_item_write_kv_pair (struct config_writer *self,
static void static void
config_item_write_object_innards config_item_write_object_innards
(struct config_writer *self, struct config_item_ *object) (struct config_writer *self, struct config_item *object)
{ {
hard_assert (object->type == CONFIG_ITEM_OBJECT); hard_assert (object->type == CONFIG_ITEM_OBJECT);
struct str_map_iter iter; struct str_map_iter iter;
str_map_iter_init (&iter, &object->value.object); str_map_iter_init (&iter, &object->value.object);
struct config_item_ *value; struct config_item *value;
while ((value = str_map_iter_next (&iter))) while ((value = str_map_iter_next (&iter)))
config_item_write_kv_pair (self, iter.link->key, value); config_item_write_kv_pair (self, iter.link->key, value);
} }
static void static void
config_item_write (struct config_item_ *value, config_item_write (struct config_item *value,
bool object_innards, struct str *output) bool object_innards, struct str *output)
{ {
struct config_writer writer = { .output = output, .indent = 0 }; struct config_writer writer = { .output = output, .indent = 0 };
@ -1769,13 +1769,13 @@ config_parser_expect
#define EXPECT(token) config_parser_expect (self, token, err) #define EXPECT(token) config_parser_expect (self, token, err)
#define SKIP_NL() do {} while (ACCEPT (CONFIG_T_NEWLINE)) #define SKIP_NL() do {} while (ACCEPT (CONFIG_T_NEWLINE))
static struct config_item_ *config_parser_parse_object static struct config_item *config_parser_parse_object
(struct config_parser *self, jmp_buf out); (struct config_parser *self, jmp_buf out);
static struct config_item_ * static struct config_item *
config_parser_parse_value (struct config_parser *self, jmp_buf out) config_parser_parse_value (struct config_parser *self, jmp_buf out)
{ {
struct config_item_ *volatile result = NULL; struct config_item *volatile result = NULL;
jmp_buf err; jmp_buf err;
if (setjmp (err)) if (setjmp (err))
@ -1810,7 +1810,7 @@ config_parser_parse_value (struct config_parser *self, jmp_buf out)
/// Parse a single "key = value" assignment into @a object /// Parse a single "key = value" assignment into @a object
static bool static bool
config_parser_parse_kv_pair (struct config_parser *self, config_parser_parse_kv_pair (struct config_parser *self,
struct config_item_ *object, jmp_buf out) struct config_item *object, jmp_buf out)
{ {
char *volatile key = NULL; char *volatile key = NULL;
jmp_buf err; jmp_buf err;
@ -1851,10 +1851,10 @@ config_parser_parse_kv_pair (struct config_parser *self,
} }
/// Parse the inside of an object definition /// Parse the inside of an object definition
static struct config_item_ * static struct config_item *
config_parser_parse_object (struct config_parser *self, jmp_buf out) config_parser_parse_object (struct config_parser *self, jmp_buf out)
{ {
struct config_item_ *volatile object = config_item_object (); struct config_item *volatile object = config_item_object ();
jmp_buf err; jmp_buf err;
if (setjmp (err)) if (setjmp (err))
@ -1877,14 +1877,14 @@ config_parser_parse_object (struct config_parser *self, jmp_buf out)
/// Parse a configuration snippet either as an object or a bare value. /// Parse a configuration snippet either as an object or a bare value.
/// If it's the latter (@a single_value_only), no newlines may follow. /// If it's the latter (@a single_value_only), no newlines may follow.
static struct config_item_ * static struct config_item *
config_item_parse (const char *script, size_t len, config_item_parse (const char *script, size_t len,
bool single_value_only, struct error **e) bool single_value_only, struct error **e)
{ {
struct config_parser parser; struct config_parser parser;
config_parser_init (&parser, script, len); config_parser_init (&parser, script, len);
struct config_item_ *volatile object = NULL; struct config_item *volatile object = NULL;
jmp_buf err; jmp_buf err;
if (setjmp (err)) if (setjmp (err))
@ -1916,14 +1916,14 @@ end:
} }
/// Clone an item. Schema assignments aren't retained. /// Clone an item. Schema assignments aren't retained.
struct config_item_ * struct config_item *
config_item_clone (struct config_item_ *self) config_item_clone (struct config_item *self)
{ {
// Oh well, it saves code // Oh well, it saves code
struct str tmp; struct str tmp;
str_init (&tmp); str_init (&tmp);
config_item_write (self, false, &tmp); config_item_write (self, false, &tmp);
struct config_item_ *result = struct config_item *result =
config_item_parse (tmp.str, tmp.len, true, NULL); config_item_parse (tmp.str, tmp.len, true, NULL);
str_free (&tmp); str_free (&tmp);
return result; return result;
@ -1933,9 +1933,9 @@ config_item_clone (struct config_item_ *self)
static void static void
config_schema_initialize_item (struct config_schema *schema, config_schema_initialize_item (struct config_schema *schema,
struct config_item_ *parent, void *user_data) struct config_item *parent, void *user_data)
{ {
struct config_item_ *item = struct config_item *item =
str_map_find (&parent->value.object, schema->name); str_map_find (&parent->value.object, schema->name);
bool replace = true; bool replace = true;
@ -1981,7 +1981,7 @@ config_schema_initialize_item (struct config_schema *schema,
static void static void
config_schema_apply_to_object (struct config_schema *schema_array, config_schema_apply_to_object (struct config_schema *schema_array,
struct config_item_ *object, void *user_data) struct config_item *object, void *user_data)
{ {
hard_assert (object->type == CONFIG_ITEM_OBJECT); hard_assert (object->type == CONFIG_ITEM_OBJECT);
while (schema_array->name) while (schema_array->name)
@ -1989,14 +1989,14 @@ config_schema_apply_to_object (struct config_schema *schema_array,
} }
static void static void
config_schema_call_changed (struct config_item_ *item) config_schema_call_changed (struct config_item *item)
{ {
if (item->type == CONFIG_ITEM_OBJECT) if (item->type == CONFIG_ITEM_OBJECT)
{ {
struct str_map_iter iter; struct str_map_iter iter;
str_map_iter_init (&iter, &item->value.object); str_map_iter_init (&iter, &item->value.object);
struct config_item_ *child; struct config_item *child;
while ((child = str_map_iter_next (&iter))) while ((child = str_map_iter_next (&iter)))
config_schema_call_changed (child); config_schema_call_changed (child);
} }
@ -2009,7 +2009,7 @@ config_schema_call_changed (struct config_item_ *item)
// XXX: this doesn't necessarily have to be well designed at all // XXX: this doesn't necessarily have to be well designed at all
typedef void (*config_module_load_fn) typedef void (*config_module_load_fn)
(struct config_item_ *subtree, void *user_data); (struct config_item *subtree, void *user_data);
struct config_module struct config_module
{ {
@ -2035,7 +2035,7 @@ config_module_destroy (struct config_module *self)
struct config struct config
{ {
struct str_map modules; ///< Toplevel modules struct str_map modules; ///< Toplevel modules
struct config_item_ *root; ///< CONFIG_ITEM_OBJECT struct config_item *root; ///< CONFIG_ITEM_OBJECT
}; };
static void static void
@ -2067,7 +2067,7 @@ config_register_module (struct config *self,
} }
static void static void
config_load (struct config *self, struct config_item_ *root) config_load (struct config *self, struct config_item *root)
{ {
hard_assert (root->type == CONFIG_ITEM_OBJECT); hard_assert (root->type == CONFIG_ITEM_OBJECT);
self->root = root; self->root = root;
@ -2078,7 +2078,7 @@ config_load (struct config *self, struct config_item_ *root)
struct config_module *module; struct config_module *module;
while ((module = str_map_iter_next (&iter))) while ((module = str_map_iter_next (&iter)))
{ {
struct config_item_ *subtree = str_map_find struct config_item *subtree = str_map_find
(&root->value.object, module->name); (&root->value.object, module->name);
// Silently fix inputs that only a lunatic user could create // Silently fix inputs that only a lunatic user could create
if (!subtree || subtree->type != CONFIG_ITEM_OBJECT) if (!subtree || subtree->type != CONFIG_ITEM_OBJECT)

90
degesch.c
View File

@ -1112,7 +1112,7 @@ struct server
char *name; ///< Server identifier char *name; ///< Server identifier
struct buffer *buffer; ///< The buffer for this server struct buffer *buffer; ///< The buffer for this server
struct config_item_ *config; ///< Configuration root struct config_item *config; ///< Configuration root
// Connection: // Connection:
@ -1473,25 +1473,25 @@ static void refresh_prompt (struct app_context *ctx);
// --- Configuration ----------------------------------------------------------- // --- Configuration -----------------------------------------------------------
static void static void
on_config_debug_mode_change (struct config_item_ *item) on_config_debug_mode_change (struct config_item *item)
{ {
g_debug_mode = item->value.boolean; g_debug_mode = item->value.boolean;
} }
static void static void
on_config_show_all_prefixes_change (struct config_item_ *item) on_config_show_all_prefixes_change (struct config_item *item)
{ {
struct app_context *ctx = item->user_data; struct app_context *ctx = item->user_data;
ctx->show_all_prefixes = item->value.boolean; ctx->show_all_prefixes = item->value.boolean;
refresh_prompt (ctx); refresh_prompt (ctx);
} }
static void on_config_attribute_change (struct config_item_ *item); static void on_config_attribute_change (struct config_item *item);
static void on_config_logging_change (struct config_item_ *item); static void on_config_logging_change (struct config_item *item);
#define TRIVIAL_BOOLEAN_ON_CHANGE(name) \ #define TRIVIAL_BOOLEAN_ON_CHANGE(name) \
static void \ static void \
on_config_ ## name ## _change (struct config_item_ *item) \ on_config_ ## name ## _change (struct config_item *item) \
{ \ { \
struct app_context *ctx = item->user_data; \ struct app_context *ctx = item->user_data; \
ctx->name = item->value.boolean; \ ctx->name = item->value.boolean; \
@ -1504,7 +1504,7 @@ TRIVIAL_BOOLEAN_ON_CHANGE (beep_on_highlight)
static bool static bool
config_validate_nonjunk_string config_validate_nonjunk_string
(const struct config_item_ *item, struct error **e) (const struct config_item *item, struct error **e)
{ {
if (item->type == CONFIG_ITEM_NULL) if (item->type == CONFIG_ITEM_NULL)
return true; return true;
@ -1525,7 +1525,7 @@ config_validate_nonjunk_string
static bool static bool
config_validate_addresses config_validate_addresses
(const struct config_item_ *item, struct error **e) (const struct config_item *item, struct error **e)
{ {
if (item->type == CONFIG_ITEM_NULL) if (item->type == CONFIG_ITEM_NULL)
return true; return true;
@ -1548,7 +1548,7 @@ config_validate_addresses
static bool static bool
config_validate_nonnegative config_validate_nonnegative
(const struct config_item_ *item, struct error **e) (const struct config_item *item, struct error **e)
{ {
if (item->type == CONFIG_ITEM_NULL) if (item->type == CONFIG_ITEM_NULL)
return true; return true;
@ -1722,13 +1722,13 @@ static struct config_schema g_config_attributes[] =
// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
static void static void
load_config_behaviour (struct config_item_ *subtree, void *user_data) load_config_behaviour (struct config_item *subtree, void *user_data)
{ {
config_schema_apply_to_object (g_config_behaviour, subtree, user_data); config_schema_apply_to_object (g_config_behaviour, subtree, user_data);
} }
static void static void
load_config_attributes (struct config_item_ *subtree, void *user_data) load_config_attributes (struct config_item *subtree, void *user_data)
{ {
config_schema_apply_to_object (g_config_attributes, subtree, user_data); config_schema_apply_to_object (g_config_attributes, subtree, user_data);
} }
@ -1747,9 +1747,9 @@ register_config_modules (struct app_context *ctx)
// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
static const char * static const char *
get_config_string (struct config_item_ *root, const char *key) get_config_string (struct config_item *root, const char *key)
{ {
struct config_item_ *item = config_item_get (root, key, NULL); struct config_item *item = config_item_get (root, key, NULL);
hard_assert (item); hard_assert (item);
if (item->type == CONFIG_ITEM_NULL) if (item->type == CONFIG_ITEM_NULL)
return NULL; return NULL;
@ -1759,12 +1759,12 @@ get_config_string (struct config_item_ *root, const char *key)
static bool static bool
set_config_string set_config_string
(struct config_item_ *root, const char *key, const char *value) (struct config_item *root, const char *key, const char *value)
{ {
struct config_item_ *item = config_item_get (root, key, NULL); struct config_item *item = config_item_get (root, key, NULL);
hard_assert (item); hard_assert (item);
struct config_item_ *new_ = config_item_string_from_cstr (value); struct config_item *new_ = config_item_string_from_cstr (value);
struct error *e = NULL; struct error *e = NULL;
if (config_item_set_from (item, new_, &e)) if (config_item_set_from (item, new_, &e))
return true; return true;
@ -1776,17 +1776,17 @@ set_config_string
} }
static int64_t static int64_t
get_config_integer (struct config_item_ *root, const char *key) get_config_integer (struct config_item *root, const char *key)
{ {
struct config_item_ *item = config_item_get (root, key, NULL); struct config_item *item = config_item_get (root, key, NULL);
hard_assert (item && item->type == CONFIG_ITEM_INTEGER); hard_assert (item && item->type == CONFIG_ITEM_INTEGER);
return item->value.integer; return item->value.integer;
} }
static bool static bool
get_config_boolean (struct config_item_ *root, const char *key) get_config_boolean (struct config_item *root, const char *key)
{ {
struct config_item_ *item = config_item_get (root, key, NULL); struct config_item *item = config_item_get (root, key, NULL);
hard_assert (item && item->type == CONFIG_ITEM_BOOLEAN); hard_assert (item && item->type == CONFIG_ITEM_BOOLEAN);
return item->value.boolean; return item->value.boolean;
} }
@ -2020,7 +2020,7 @@ log_message_attributed (void *user_data, const char *quote, const char *fmt,
} }
static void static void
apply_attribute_change (struct config_item_ *item, int id) apply_attribute_change (struct config_item *item, int id)
{ {
struct app_context *ctx = item->user_data; struct app_context *ctx = item->user_data;
free (ctx->attrs[id]); free (ctx->attrs[id]);
@ -2030,7 +2030,7 @@ apply_attribute_change (struct config_item_ *item, int id)
} }
static void static void
on_config_attribute_change (struct config_item_ *item) on_config_attribute_change (struct config_item *item)
{ {
static const char *table[ATTR_COUNT] = static const char *table[ATTR_COUNT] =
{ {
@ -3055,7 +3055,7 @@ buffer_close_log_file (struct buffer *buffer)
} }
static void static void
on_config_logging_change (struct config_item_ *item) on_config_logging_change (struct config_item *item)
{ {
struct app_context *ctx = item->user_data; struct app_context *ctx = item->user_data;
ctx->logging = item->value.boolean; ctx->logging = item->value.boolean;
@ -6737,11 +6737,11 @@ struct config_dump_data
}; };
static void config_dump_item static void config_dump_item
(struct config_item_ *item, struct config_dump_data *data); (struct config_item *item, struct config_dump_data *data);
static void static void
config_dump_children config_dump_children
(struct config_item_ *object, struct config_dump_data *data) (struct config_item *object, struct config_dump_data *data)
{ {
hard_assert (object->type == CONFIG_ITEM_OBJECT); hard_assert (object->type == CONFIG_ITEM_OBJECT);
@ -6754,7 +6754,7 @@ config_dump_children
struct str_map_iter iter; struct str_map_iter iter;
str_map_iter_init (&iter, &object->value.object); str_map_iter_init (&iter, &object->value.object);
struct config_item_ *child; struct config_item *child;
while ((child = str_map_iter_next (&iter))) while ((child = str_map_iter_next (&iter)))
{ {
level.name = iter.link->key; level.name = iter.link->key;
@ -6766,7 +6766,7 @@ config_dump_children
} }
static void static void
config_dump_item (struct config_item_ *item, struct config_dump_data *data) config_dump_item (struct config_item *item, struct config_dump_data *data)
{ {
// Empty objects will show as such // Empty objects will show as such
if (item->type == CONFIG_ITEM_OBJECT if (item->type == CONFIG_ITEM_OBJECT
@ -6817,7 +6817,7 @@ config_dump_item (struct config_item_ *item, struct config_dump_data *data)
} }
static void static void
config_dump (struct config_item_ *root, struct str_vector *output) config_dump (struct config_item *root, struct str_vector *output)
{ {
struct config_dump_data data; struct config_dump_data data;
data.head = NULL; data.head = NULL;
@ -6908,7 +6908,7 @@ check_server_name_for_addition (struct app_context *ctx, const char *name)
static struct server * static struct server *
server_add (struct app_context *ctx, server_add (struct app_context *ctx,
const char *name, struct config_item_ *subtree) const char *name, struct config_item *subtree)
{ {
hard_assert (!str_map_find (&ctx->servers, name)); hard_assert (!str_map_find (&ctx->servers, name));
@ -6947,7 +6947,7 @@ server_add_new (struct app_context *ctx, const char *name)
// of manual edits to the configuration, though, and they're not really // of manual edits to the configuration, though, and they're not really
// going to break anything. They only cause surprises when loading. // going to break anything. They only cause surprises when loading.
struct str_map *servers = get_servers_config (ctx); struct str_map *servers = get_servers_config (ctx);
struct config_item_ *subtree = config_item_object (); struct config_item *subtree = config_item_object ();
str_map_set (servers, name, subtree); str_map_set (servers, name, subtree);
struct server *s = server_add (ctx, name, subtree); struct server *s = server_add (ctx, name, subtree);
@ -7259,7 +7259,7 @@ handle_command_buffer (struct handler_args *a)
static bool static bool
replace_string_array replace_string_array
(struct config_item_ *item, struct str_vector *array, struct error **e) (struct config_item *item, struct str_vector *array, struct error **e)
{ {
char *changed = join_str_vector (array, ','); char *changed = join_str_vector (array, ',');
struct str tmp = { .str = changed, .len = strlen (changed) }; struct str tmp = { .str = changed, .len = strlen (changed) };
@ -7271,7 +7271,7 @@ replace_string_array
static bool static bool
handle_command_set_add handle_command_set_add
(struct config_item_ *item, const char *value, struct error **e) (struct config_item *item, const char *value, struct error **e)
{ {
bool result = false; bool result = false;
struct str_vector items; struct str_vector items;
@ -7294,7 +7294,7 @@ handle_command_set_add
static bool static bool
handle_command_set_remove handle_command_set_remove
(struct config_item_ *item, const char *value, struct error **e) (struct config_item *item, const char *value, struct error **e)
{ {
bool result = false; bool result = false;
struct str_vector items; struct str_vector items;
@ -7318,9 +7318,9 @@ handle_command_set_remove
static void static void
handle_command_set_assign_item (struct app_context *ctx, handle_command_set_assign_item (struct app_context *ctx,
char *key, struct config_item_ *new_, bool add, bool remove) char *key, struct config_item *new_, bool add, bool remove)
{ {
struct config_item_ *item = struct config_item *item =
config_item_get (ctx->config.root, key, NULL); config_item_get (ctx->config.root, key, NULL);
hard_assert (item); hard_assert (item);
@ -7369,7 +7369,7 @@ handle_command_set_assign
return false; return false;
struct error *e = NULL; struct error *e = NULL;
struct config_item_ *new_ = struct config_item *new_ =
config_item_parse (arguments, strlen (arguments), true, &e); config_item_parse (arguments, strlen (arguments), true, &e);
if (e) if (e)
{ {
@ -7447,7 +7447,7 @@ show_aliases_list (struct app_context *ctx)
struct str_map_iter iter; struct str_map_iter iter;
str_map_iter_init (&iter, aliases); str_map_iter_init (&iter, aliases);
struct config_item_ *alias; struct config_item *alias;
while ((alias = str_map_iter_next (&iter))) while ((alias = str_map_iter_next (&iter)))
{ {
struct str definition; struct str definition;
@ -7473,7 +7473,7 @@ handle_command_alias (struct handler_args *a)
if (!*a->arguments) if (!*a->arguments)
return false; return false;
struct config_item_ *alias = config_item_string_from_cstr (a->arguments); struct config_item *alias = config_item_string_from_cstr (a->arguments);
struct str definition; struct str definition;
str_init (&definition); str_init (&definition);
@ -8263,7 +8263,7 @@ g_command_handlers[] =
static bool static bool
try_handle_command_help_option (struct app_context *ctx, const char *name) try_handle_command_help_option (struct app_context *ctx, const char *name)
{ {
struct config_item_ *item = struct config_item *item =
config_item_get (ctx->config.root, name, NULL); config_item_get (ctx->config.root, name, NULL);
if (!item) if (!item)
return false; return false;
@ -8465,7 +8465,7 @@ static bool
expand_alias (struct app_context *ctx, expand_alias (struct app_context *ctx,
const char *alias_name, char *input, struct str_vector *commands) const char *alias_name, char *input, struct str_vector *commands)
{ {
struct config_item_ *entry = struct config_item *entry =
str_map_find (get_aliases_config (ctx), alias_name); str_map_find (get_aliases_config (ctx), alias_name);
if (!entry) if (!entry)
return false; return false;
@ -8715,7 +8715,7 @@ complete_command (struct app_context *ctx, struct completion *data,
struct str_map_iter iter; struct str_map_iter iter;
str_map_iter_init (&iter, get_aliases_config (ctx)); str_map_iter_init (&iter, get_aliases_config (ctx));
struct config_item_ *alias; struct config_item *alias;
while ((alias = str_map_iter_next (&iter))) while ((alias = str_map_iter_next (&iter)))
{ {
if (!strncasecmp_ascii (word, iter.link->key, word_len)) if (!strncasecmp_ascii (word, iter.link->key, word_len))
@ -9470,10 +9470,10 @@ read_file (const char *filename, struct str *output, struct error **e)
return false; return false;
} }
static struct config_item_ * static struct config_item *
load_configuration_file (const char *filename, struct error **e) load_configuration_file (const char *filename, struct error **e)
{ {
struct config_item_ *root = NULL; struct config_item *root = NULL;
struct str data; struct str data;
str_init (&data); str_init (&data);
@ -9544,7 +9544,7 @@ load_default_aliases (struct app_context *ctx)
static void static void
load_configuration (struct app_context *ctx) load_configuration (struct app_context *ctx)
{ {
struct config_item_ *root = NULL; struct config_item *root = NULL;
struct error *e = NULL; struct error *e = NULL;
char *filename = resolve_filename char *filename = resolve_filename
@ -9587,7 +9587,7 @@ load_servers (struct app_context *ctx)
struct str_map_iter iter; struct str_map_iter iter;
str_map_iter_init (&iter, get_servers_config (ctx)); str_map_iter_init (&iter, get_servers_config (ctx));
struct config_item_ *subtree; struct config_item *subtree;
while ((subtree = str_map_iter_next (&iter))) while ((subtree = str_map_iter_next (&iter)))
{ {
const char *name = iter.link->key; const char *name = iter.link->key;