Question about allowed function call

[LinusU]

Should this style of function calling be allowed? I haven't seen it anywhere in the wild until a coworker tried merging it to one of our repos today...

window.addEventListener
  ('keydown', myKeydownHandler, false)

I feel that the allowed options should be

window.addEventListener('keydown', myKeydownHandler, false)

,

window.addEventListener(
  'keydown',
  myKeydownHandler,
  false
)

and possibly

window.addEventListener(
  'keydown', myKeydownHandler, false
)

as this is what I've seen being used by the community the most.

[yoshuawuyts]

window.addEventListener
  ('keydown', myKeydownHandler, false)

seems definitely off. All the other ones feel like a bit of a smell to me, but sure - there's probably a valid use case for them

[timoxley]

I use the others on occasion, but parens on a new line seems bad

[feross]

Agreed. The paren should not go on a separate line. I don't think there is an eslint rule for this.

Does someone want to volunteer to open an issue / send a PR to eslint to add this?

[dcousens]

Agreed, that is incredibly confusing.
I use the others on occasion, but never parens on a new line.

[feross]

This will be fixed in standard v8! I just replaced the deprecated "no-spaced-func" rule with "func-call-spacing" which checks for this!

Commit: standard/eslint-config-standard@f419de5

Rule: http://eslint.org/docs/rules/func-call-spacing