wash/ivy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

lang: ast: fix selector to_string not including keyword message labels

Browse Source
This commit is contained in:
max 🍓
2024-12-06 22:25:33 +00:00
parent dc55cfa1f8
commit 68ad0655aa
1 changed files with 33 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
lang/ast/selector.c
View File

@@ -93,7 +93,8 @@ static struct token_parse_result parse_label(
struct ivy_ast_selector_node *sel struct ivy_ast_selector_node *sel
= (struct ivy_ast_selector_node *)state->s_base.s_node; = (struct ivy_ast_selector_node *)state->s_base.s_node;
if (sel->n_recipient == IVY_SELECTOR_RECIPIENT_NONE && state->s_prev != IVY_TOK_IDENT if (sel->n_recipient == IVY_SELECTOR_RECIPIENT_NONE
&& state->s_prev != IVY_TOK_IDENT
&& state->s_prev != IVY_SYM_LEFT_PAREN) { && state->s_prev != IVY_SYM_LEFT_PAREN) {
/* if recipient is not NONE, this selector appears at the beginning of a message /* if recipient is not NONE, this selector appears at the beginning of a message
* handler. only then is a label without a preceding identifier allowed. */ * handler. only then is a label without a preceding identifier allowed. */
@@ -180,7 +181,8 @@ static struct token_parse_result parse_pipe(
switch (state->s_prev) { switch (state->s_prev) {
case IVY_SYM_RIGHT_PAREN: case IVY_SYM_RIGHT_PAREN:
/* this looks like a complex message selector */ /* this looks like a complex message selector */
if (b_queue_empty(&sel->n_arg_labels) || b_queue_empty(&sel->n_arg_names)) { if (b_queue_empty(&sel->n_arg_labels)
|| b_queue_empty(&sel->n_arg_names)) {
/* no message args */ /* no message args */
return PARSE_RESULT(IVY_ERR_BAD_SYNTAX, 0); return PARSE_RESULT(IVY_ERR_BAD_SYNTAX, 0);
} }
@@ -192,7 +194,8 @@ static struct token_parse_result parse_pipe(
break; break;
case IVY_TOK_IDENT: case IVY_TOK_IDENT:
/* this looks like a unary or keyword message. */ /* this looks like a unary or keyword message. */
if (!b_queue_empty(&sel->n_arg_labels) && b_queue_empty(&sel->n_arg_names)) { if (!b_queue_empty(&sel->n_arg_labels)
&& b_queue_empty(&sel->n_arg_names)) {
/* keyword message with no arg names. */ /* keyword message with no arg names. */
return PARSE_RESULT(IVY_ERR_BAD_SYNTAX, 0); return PARSE_RESULT(IVY_ERR_BAD_SYNTAX, 0);
} }
@@ -224,7 +227,8 @@ static void to_string(struct ivy_ast_node *node, b_string *str)
{ {
struct ivy_ast_selector_node *sel = (struct ivy_ast_selector_node *)node; struct ivy_ast_selector_node *sel = (struct ivy_ast_selector_node *)node;
b_string_append_cstrf(str, "%s [", ivy_ast_node_type_to_string(node->n_type)); b_string_append_cstrf(
str, "%s [", ivy_ast_node_type_to_string(node->n_type));
switch (sel->n_recipient) { switch (sel->n_recipient) {
case IVY_SELECTOR_RECIPIENT_CLASS: case IVY_SELECTOR_RECIPIENT_CLASS:
@@ -255,22 +259,28 @@ static void to_string(struct ivy_ast_node *node, b_string *str)
b_queue_iterator_begin(&sel->n_arg_names, &name_it); b_queue_iterator_begin(&sel->n_arg_names, &name_it);
int i = 0; int i = 0;
while (b_queue_iterator_is_valid(&label_it) while (b_queue_iterator_is_valid(&label_it)) {
&& b_queue_iterator_is_valid(&name_it)) {
if (i > 0) { if (i > 0) {
b_string_append_cstr(str, " "); b_string_append_cstr(str, " ");
} }
struct ivy_token *label struct ivy_token *label, *name;
= b_unbox(struct ivy_token, label_it.entry, t_entry);
struct ivy_token *name
= b_unbox(struct ivy_token, name_it.entry, t_entry);
if (label && name) { label = b_unbox(struct ivy_token, label_it.entry, t_entry);
b_string_append_cstrf(str, "%s:%s", label->t_str, name->t_str); name = b_unbox(struct ivy_token, name_it.entry, t_entry);
if (label) {
b_string_append_cstrf(str, "%s:", label->t_str);
} }
if (name) {
b_string_append_cstrf(str, "%s", name->t_str);
}
if (name) {
i++; i++;
}
b_queue_iterator_next(&label_it); b_queue_iterator_next(&label_it);
b_queue_iterator_next(&name_it); b_queue_iterator_next(&name_it);
} }