Skip to content

Commit 3665d0b

Browse files
committed
py: Simplify some inline-assembler error messages, but retain meaning.
Just to reduce code size. Messages are still to the point and unambiguous.
1 parent 9c5cabb commit 3665d0b

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

py/compile.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3389,7 +3389,7 @@ STATIC void compile_scope_inline_asm(compiler_t *comp, scope_t *scope, pass_kind
33893389
} else if (MP_PARSE_NODE_STRUCT_KIND(pns2) != PN_expr_stmt) {
33903390
// not an instruction; error
33913391
not_an_instruction:
3392-
compile_syntax_error(comp, nodes[i], "inline assembler expecting an instruction");
3392+
compile_syntax_error(comp, nodes[i], "expecting an assembler instruction");
33933393
return;
33943394
}
33953395

@@ -3420,7 +3420,7 @@ STATIC void compile_scope_inline_asm(compiler_t *comp, scope_t *scope, pass_kind
34203420
// emit instructions
34213421
if (op == MP_QSTR_label) {
34223422
if (!(n_args == 1 && MP_PARSE_NODE_IS_ID(pn_arg[0]))) {
3423-
compile_syntax_error(comp, nodes[i], "inline assembler 'label' requires 1 argument");
3423+
compile_syntax_error(comp, nodes[i], "'label' requires 1 argument");
34243424
return;
34253425
}
34263426
uint lab = comp_next_label(comp);
@@ -3432,22 +3432,22 @@ STATIC void compile_scope_inline_asm(compiler_t *comp, scope_t *scope, pass_kind
34323432
}
34333433
} else if (op == MP_QSTR_align) {
34343434
if (!(n_args == 1 && MP_PARSE_NODE_IS_SMALL_INT(pn_arg[0]))) {
3435-
compile_syntax_error(comp, nodes[i], "inline assembler 'align' requires 1 argument");
3435+
compile_syntax_error(comp, nodes[i], "'align' requires 1 argument");
34363436
return;
34373437
}
34383438
if (pass > MP_PASS_SCOPE) {
34393439
EMIT_INLINE_ASM_ARG(align, MP_PARSE_NODE_LEAF_SMALL_INT(pn_arg[0]));
34403440
}
34413441
} else if (op == MP_QSTR_data) {
34423442
if (!(n_args >= 2 && MP_PARSE_NODE_IS_SMALL_INT(pn_arg[0]))) {
3443-
compile_syntax_error(comp, nodes[i], "inline assembler 'data' requires at least 2 arguments");
3443+
compile_syntax_error(comp, nodes[i], "'data' requires at least 2 arguments");
34443444
return;
34453445
}
34463446
if (pass > MP_PASS_SCOPE) {
34473447
mp_int_t bytesize = MP_PARSE_NODE_LEAF_SMALL_INT(pn_arg[0]);
34483448
for (uint j = 1; j < n_args; j++) {
34493449
if (!MP_PARSE_NODE_IS_SMALL_INT(pn_arg[j])) {
3450-
compile_syntax_error(comp, nodes[i], "inline assembler 'data' requires integer arguments");
3450+
compile_syntax_error(comp, nodes[i], "'data' requires integer arguments");
34513451
return;
34523452
}
34533453
EMIT_INLINE_ASM_ARG(data, bytesize, MP_PARSE_NODE_LEAF_SMALL_INT(pn_arg[j]));

0 commit comments

Comments
 (0)