bug#9151: js-mode indentation doesn't recognize brace-less function expressions

bug#9151: js-mode indentation doesn't recognize brace-less function expressions

[Štěpán Němec]

GNU Emacs 24.0.50.6 (i686-pc-linux-gnu) of 2011-01-15 on headley

In JavaScript, the following is a valid function definition:

function foo ()
    a.somewhat().complicated()
    .computation()

but `js-mode' currently indents it like this:

function foo ()
a.somewhat().complicated()
    .computation()

-- 
Štěpán

bug#9151: js-mode indentation doesn't recognize brace-less function expressions

[Knut Olav Bøhmer]

[-- Attachment #1: Type: text/plain, Size: 405 bytes --]

This bug is probably more important to day then 8 years ago because of the
widely used  arrow function with concise body syntax as documented here:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions#Function_body

Right:
const something = () =>
      indent_like_this();

Wrong:
const something = () =>
not_indent_like_this();

-- 
Knut Olav Bøhmer

[-- Attachment #2: Type: text/html, Size: 704 bytes --]

bug#9151: js-mode indentation doesn't recognize brace-less function expressions

[Jakub Ječmínek via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Knut Olav Bøhmer <bohmer@gmail.com> writes:

> Right:
> const something = () =>
>       indent_like_this();
>
> Wrong:
> const something = () =>
> not_indent_like_this();

This indents correctly for me.


Štěpán Němec <stepnem@gmail.com> writes:

> In JavaScript, the following is a valid function definition:
>
> function foo ()
>     a.somewhat().complicated()
>     .computation()
>
> but `js-mode' currently indents it like this:
>
> function foo ()
> a.somewhat().complicated()
>     .computation()

I know virtually nothing about JavaScript but your example function
doesn't seem valid.  If you add curly braces and set `js-chain-indent'
to t then it works fine.