Skip to content

Throw syntax error on a function call line which ends with a comma and further line(s) without indentation  #1108

@ceremcem

Description

@ceremcem

I've came across with this case numerous times. Consider the following example:

hello foo, bar
hi there

Compiled output is obviously:

hello(foo, bar);
hi(there);

Then I remove the bar parameter, but accidentally forgot to remove the comma. The compiled output becomes:

hello(foo, hi(there));

instead of:

hello(foo);
hi(there);

Proposal

I guess this is not a bug, but a feature. However, I think LiveScript will be more error resistant if this feature is removed and users are forced to use backslash or parenthesis for multi line splitting purposes.

At least, an indentation for further lines must be required for considering a line to be split. to consider the line split.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions