Ternary operator and self-executing function

[Zenwolf]

This issue demonstrates two problems:

1. A case where a self-executing function is not considered a function call when used in a ternary operator.
2. A semicolon is considered unnecessary when it is defensive in nature.

PROBLEM 1: When I add a self-executing function as part of a ternary operator expression, I get an error.

Error received from jshint:
Expected an assignment or function call and instead saw an expression. "} ())"

A self-executing function is a function call: it calls itself, and it should be treated as a function call.

EXAMPLE:

var foo;

; (foo)
? foo.bar = {}
: (function () {    
    foo = {};
    foo.bar = {};
} ())
;

PROBLEM 2: Semicolon before expression parentheses.

The initial semicolon above is reported as unnecessary, but can help prevent errors from code that may have been included that did not include appropriate semicolons.

Thanks!

[Zenwolf]

Actually, this function error still happens when I break the function call out into a standard function:

var foo;

function myFunc() {
    foo = {};
    foo.bar = {};
}

; (foo)
? foo.bar = {}
: myFunc()
;

[Zenwolf]

closed by mistake.

[ryanfitzer]

This also exists when using other operators to create an immediately invoked function expression:

!function() {

}();

[goatslacker]

@ryanfitzer you can turn that on by setting expr flag to true.

[ryanfitzer]

That was it! Thanks soo much.

Any clue on the "Missing space after 'function'." warning? This happens if there is no space between the word function and ()?

!function() {}();

I assumed it was the white option, but that wasn't the case. I figure it is checking it against the same rules that apply to if/while/for type expressions, but I can't find the option (if it even exists).

Thanks again.

[goatslacker]

It's supposed to be white.

This works for me:

/*jshint expr: true, white: false */

!function() {
}();

[ryanfitzer]

Just tried yours and it worked. Not sure why, but indent: 4 is causing the issue. When removed, it works?

Not sure why it would conflict. Bug?

[WolfgangKluge]

indent implies white: true... Hope we can remove it someday (or output a warning at least)

[valueof]

This was fixed in GH-414.