IIFE changed to Crockford's prefered format

[yitsushi]

IIFE changed to Crockford's prefered format.

The first parenthesis is a marker for an IIFE and there is no need to call the function only for let JavaScript know it will be called immediately.

Some linter warns for it like jshint (http://www.jslint.com/).
"Move the invocation into the parens that contain the function."

Function Declarations part of http://javascript.crockford.com/code.html

[serbanghita]

Agree, good catch!

[hshoff]

Hey @yitsushi, thanks for checking out our style guide!

This is a style choice. The most important part is consistency.

At Airbnb we prefer !function(){}(); if we're wrapping a module (reasoning). I wrote our style guide's IIFE in the style of (function(){})(); because that was how a lot of IIFEs in our codebase were written.

Feel free to fork the guide and update it with crockford's prefered format! We'd be happy to link to it in the resources.

🍻

[yitsushi]

+1

:) And jshint (or similar) does not warn you? But, yes I understand :)

[hshoff]

We use these jshint settings: https://github.com/airbnb/javascript/blob/master/linters/SublimeLinter/SublimeLinter.sublime-settings