Consistent use the same IIFE style in javascript
authorTimo Tijhof <krinklemail@gmail.com>
Fri, 13 Dec 2013 18:53:28 +0000 (19:53 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Wed, 30 Apr 2014 15:09:24 +0000 (17:09 +0200)
Most of our IIFE (Immediately-Invoced Function Expression) were
already the same way, these two were different. Made them
consistent (parenthesis on the outside).

$ ack '\}\s*\([^)]+\)\s*\)' --js resources/src/ skins/
  81 matches (0 false positives)

$ ack '\}\s*\)\s*\(' --js resources/src/ skins/
  6 matches (2 false positives, 2 in upstream forks)

Fixed the two matches in the last one that weren't upstream forks.

Change-Id: I51da0e7eb3c0a7071af0ebaa3310e31c99b9d3a5

resources/src/mediawiki.page/mediawiki.page.gallery.js
resources/src/mediawiki/mediawiki.util.js

index 671cec6..e7c962f 100644 (file)
                                                }
                                        }
                                }
-                       } )();
+                       }() );
                } );
        } );
-} )( jQuery, mediaWiki );
+}( jQuery, mediaWiki ) );
index eb20d22..f09d03c 100644 (file)
@@ -57,7 +57,7 @@
 
                                // Make sure we don't unset util.$content if it was preset and we don't find anything
                                return util.$content;
-                       } )();
+                       }() );
                },
 
                /* Main body */