Semi-colons do not belong at the end of function declarations about box2dweb HOT 6 OPEN

Hi,

Is there a good source which confirms your claim?

If I remember correctly, I had some problems with minification when removing 
them, but I'm not sure. Furthermore I read somewhere that they belong there, 
but came to the conclusion that they don't make sense.

http://www.jslint.com/

should hopefully provide an answer.  Minification often requires extra 
semicolons to work flawlessly (because it gets rid of whitespace including 
times where ECMAscript treats a newline differently than a space.  Extra ;'s 
aren't going to hurt anything but if you want syntactical perfection the domain 
above is your definitive guide.

It's not "[my] claim" - it's the syntactic ASI rules of JavaScript as set forth 
by the ECMAScript spec - dating back to ES1. And yes, JSLint will confirm this, 
eg.

    function foo() {
        return true;
    };            

JSLint Result:

    Problem at line 3 character 2: Unexpected ';'.


As far as minification is concerned, the only expectation is that semi-colons 
are inserted correctly and you'll have no issues. The problem years ago 
centered around concatenating files from varying sources with varying 
reliability.

Yes, JSLint is a perfect source and confirms it. Maybe, "claim" was the wrong 
word to choose ;)

I would recommend a slightly modified function declaration style which is 
correct to add a semi-colon at the end, I believe it is a bit cleaner (and 
shorter than the style you have now):

change:
   function b2Body() {
      ...
   };
   Box2D.Dynamics.b2Body = b2Body;

to:
   Box2D.Dynamics.b2Body = function () {
      ...
   };

I'm working on a new version of my compiler at the moment which will not 
produce such a code.