Re: [PATCH] pkl: parser: make parens optional for format

[Jose E. Marchesi]

> +        | FORMAT STR format_arg_list
> +                {
> +                  $$ = pkl_ast_make_format (pkl_parser->ast, $2, $3,
> +                                            0 /* printf_p */);
> +                  PKL_AST_TYPE ($$)
> +                      = ASTREF (pkl_ast_make_string_type (pkl_parser->ast));

Now that I notice, is there any reason why the type of the format AST
node is not set in typify1 like every other node?  Same for the other
syntax.

> +                  PKL_AST_LOC ($2) = @2;
> +                  if (PKL_AST_TYPE ($2))
> +                    PKL_AST_LOC (PKL_AST_TYPE ($2)) = @2;

That location is no longer necesary.

> +                  PKL_AST_LOC ($$) = @$;
> +                  PKL_AST_LOC (PKL_AST_TYPE ($$)) = @$;

Likewise.

> +                }