The empty of functions

[nodtem66]

Current usage of empty for function is:

empty(function(){}); // true
empty(function(a,b){}); // false

then empty(function() { return 1; }); should be false logically, but it failed (see: Runkit).

According to the jfriend00's answer, coding to check the empty of function's body is no totally reliable.
Shouldn't the empty of all functions be false, like the empty of all boolean?

[ianstormtaylor]

Hey @nodtem66, sorry for the confusion here. The "emptiness" of functions that I originally added was for checking function.length which checks if the function takes no arguments, not if it has an empty function body.

I would agree that it's probably not the most useful of the cases though.

I probably wouldn't want to major bump right now to remove it, but I could be down to consider removing that piece in the future.

Thanks!

[Eli-Black-Work]

This seems confusing to me, too. I'd expect empty() to return false for all functions.