Use designated initializers

Author: amomchilov

src/ast.c

@@ -252,9 +252,11 @@ void rbs_hash_set(rbs_hash_t *hash, rbs_node_t *key, rbs_node_t *value) {
     }
 
     rbs_hash_node_t *new_node = rbs_alloc(hash->allocator, rbs_hash_node_t);
-    new_node->key = key;
-    new_node->value = value;
-    new_node->next = NULL;
+    *new_node = (rbs_hash_node_t) {
+        .key = key,
+        .value = value,
+        .next = NULL,
+    };
 
     if (hash->tail == NULL) {
         hash->head = new_node;

src/location.c

@@ -10,18 +10,24 @@ void rbs_loc_alloc_children(rbs_allocator_t *allocator, rbs_location_t *loc, siz
 
     loc->children = rbs_allocator_malloc_impl(allocator, RBS_LOC_CHILDREN_SIZE(capacity), rbs_alignof(rbs_loc_children));
 
-    loc->children->len = 0;
-    loc->children->required_p = 0;
-    loc->children->cap = capacity;
+    *loc->children = (rbs_loc_children) {
+        .len = 0,
+        .cap = capacity,
+        .required_p = 0,
+        .entries = { 0 },
+    };
 }
 
 void rbs_loc_add_optional_child(rbs_location_t *loc, rbs_constant_id_t name, rbs_range_t r) {
     rbs_assert(loc->children != NULL, "All children should have been pre-allocated with rbs_loc_alloc_children()");
     rbs_assert((loc->children->len + 1 <= loc->children->cap), "Not enough space was pre-allocated for the children. Children: %hu, Capacity: %hu", loc->children->len, loc->children->cap);
 
     unsigned short i = loc->children->len++;
-    loc->children->entries[i].name = name;
-    loc->children->entries[i].rg = (rbs_loc_range) { r.start.char_pos, r.end.char_pos };
+
+    loc->children->entries[i] = (rbs_loc_entry) {
+        .name = name,
+        .rg = (rbs_loc_range) { r.start.char_pos, r.end.char_pos },
+    };
 }
 
 void rbs_loc_add_required_child(rbs_location_t *loc, rbs_constant_id_t name, rbs_range_t r) {

src/parser.c

@@ -837,9 +837,12 @@ static bool parse_function(rbs_parser_t *parser, bool accept_type_binding, parse
         );
     }
 
-    (*result)->function = function;
-    (*result)->block = block;
-    (*result)->function_self_type = function_self_type;
+    **result = (parse_function_result) {
+        .function = function,
+        .block = block,
+        .function_self_type = function_self_type,
+    };
+
     return true;
 }
 
@@ -1787,8 +1790,10 @@ static bool parse_method_name(rbs_parser_t *parser, rbs_range_t *range, rbs_ast_
     case tULLIDENT:
         KEYWORD_CASES
         if (parser->next_token.type == pQUESTION && parser->current_token.range.end.byte_pos == parser->next_token.range.start.byte_pos) {
-            range->start = parser->current_token.range.start;
-            range->end = parser->next_token.range.end;
+            *range = (rbs_range_t) {
+                .start = parser->current_token.range.start,
+                .end = parser->next_token.range.end,
+            };
             rbs_parser_advance(parser);
 
             rbs_constant_id_t constant_id = rbs_constant_pool_insert_shared_with_encoding(
@@ -3554,9 +3559,11 @@ void rbs_parser_set_error(rbs_parser_t *parser, rbs_token_t tok, bool syntax_err
     va_end(args);
 
     parser->error = rbs_alloc(ALLOCATOR(), rbs_error_t);
-    parser->error->token = tok;
-    parser->error->message = message;
-    parser->error->syntax_error = syntax_error;
+    *parser->error = (rbs_error_t) {
+        .message = message,
+        .token = tok,
+        .syntax_error = syntax_error,
+    };
 }
 
 /*

templates/src/ast.c.erb

@@ -105,9 +105,11 @@ void rbs_hash_set(rbs_hash_t *hash, rbs_node_t *key, rbs_node_t *value) {
     }
 
     rbs_hash_node_t *new_node = rbs_alloc(hash->allocator, rbs_hash_node_t);
-    new_node->key = key;
-    new_node->value = value;
-    new_node->next = NULL;
+    *new_node = (rbs_hash_node_t) {
+        .key = key,
+        .value = value,
+        .next = NULL,
+    };
 
     if (hash->tail == NULL) {
         hash->head = new_node;