From f7ae48637fb76900968110014cd7b23c961acbbc Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Mon, 9 Sep 2013 21:58:09 -0500
Subject: [PATCH 1/9] updates

---
 README.md | 271 +++++++++++++++++++++++++-----------------------------
 1 file changed, 124 insertions(+), 147 deletions(-)

diff --git a/README.md b/README.md
index dbafdc488d..88bd68d740 100644
--- a/README.md
+++ b/README.md
@@ -47,8 +47,8 @@
     + `undefined`
 
     ```javascript
-    var foo = 1,
-        bar = foo;
+    var foo = 1;
+    var bar = foo;
 
     bar = 9;
 
@@ -61,8 +61,8 @@
     + `function`
 
     ```javascript
-    var foo = [1, 2],
-        bar = foo;
+    var foo = [1, 2];
+    var bar = foo;
 
     bar[0] = 9;
 
@@ -163,7 +163,7 @@
   - To convert an array-like object to an array, use Array#slice.
 
     ```javascript
-    function trigger() {
+    function trigger () {
       var args = Array.prototype.slice.call(arguments);
       ...
     }
@@ -218,11 +218,8 @@
   - When programatically building up a string, use Array#join instead of string concatenation. Mostly for IE: [jsPerf](http://jsperf.com/string-vs-array-concat/2).
 
     ```javascript
-    var items,
-        messages,
-        length, i;
 
-    messages = [{
+    var messages = [{
         state: 'success',
         message: 'This one worked.'
     },{
@@ -233,11 +230,12 @@
         message: 'This one did not work.'
     }];
 
-    length = messages.length;
+    var length = messages.length;
 
     // bad
-    function inbox(messages) {
-      items = '<ul>';
+    function inbox (messages) {
+      var items = '<ul>';
+      var i;
 
       for (i = 0; i < length; i++) {
         items += '<li>' + messages[i].message + '</li>';
@@ -247,8 +245,9 @@
     }
 
     // good
-    function inbox(messages) {
-      items = [];
+    function inbox (messages) {
+      var items = [];
+      var i;
 
       for (i = 0; i < length; i++) {
         items[i] = messages[i].message;
@@ -267,17 +266,17 @@
 
     ```javascript
     // anonymous function expression
-    var anonymous = function() {
+    var anonymous = function () {
       return true;
     };
 
     // named function expression
-    var named = function named() {
+    var named = function named () {
       return true;
     };
 
     // immediately-invoked function expression (IIFE)
-    (function() {
+    (function () {
       console.log('Welcome to the Internet. Please follow me.');
     })();
     ```
@@ -288,14 +287,14 @@
     ```javascript
     // bad
     if (currentUser) {
-      function test() {
+      function test () {
         console.log('Nope.');
       }
     }
 
     // good
     if (currentUser) {
-      var test = function test() {
+      var test = function test () {
         console.log('Yup.');
       };
     }
@@ -305,12 +304,12 @@
 
     ```javascript
     // bad
-    function nope(name, options, arguments) {
+    function nope (name, options, arguments) {
       // ...stuff...
     }
 
     // good
-    function yup(name, options, args) {
+    function yup (name, options, args) {
       // ...stuff...
     }
     ```
@@ -344,7 +343,7 @@
       age: 28
     };
 
-    function getProp(prop) {
+    function getProp (prop) {
       return luke[prop];
     }
 
@@ -366,47 +365,50 @@
     var superPower = new SuperPower();
     ```
 
-  - Use one `var` declaration for multiple variables and declare each variable on a newline.
+  - Use a `var` declaration for each variable and declare each variable on a newline.
 
     ```javascript
     // bad
-    var items = getItems();
-    var goSportsTeam = true;
-    var dragonball = 'z';
-
-    // good
     var items = getItems(),
         goSportsTeam = true,
         dragonball = 'z';
+
+    // good
+    var items = getItems();
+    var goSportsTeam = true;
+    var dragonball = 'z';
     ```
 
   - Declare unassigned variables last. This is helpful when later on you might need to assign a variable depending on one of the previous assigned variables.
 
     ```javascript
     // bad
-    var i, len, dragonball,
-        items = getItems(),
-        goSportsTeam = true;
+    var i;
+    var len;
+    var dragonball;
+    var items = getItems();
+    var goSportsTeam = true;
 
     // bad
-    var i, items = getItems(),
-        dragonball,
-        goSportsTeam = true,
-        len;
+    var i;
+    var items = getItems();
+    var dragonball;
+    var goSportsTeam = true;
+    var len;
 
     // good
-    var items = getItems(),
-        goSportsTeam = true,
-        dragonball,
-        length,
-        i;
+    var items = getItems();
+    var goSportsTeam = true;
+    var dragonball;
+    var length;
+    var i;
     ```
 
-  - Assign variables at the top of their scope. This helps avoid issues with variable declaration and assignment hoisting related issues.
+  - Assign variables at the top of their scope when not inconvenient. This helps avoid issues with variable declaration and assignment hoisting related issues.
 
     ```javascript
     // bad
-    function() {
+    function () {
       test();
       console.log('doing stuff..');
 
@@ -422,7 +424,7 @@
     }
 
     // good
-    function() {
+    function () {
       var name = getName();
 
       test();
@@ -438,7 +440,7 @@
     }
 
     // bad
-    function() {
+    function () {
       var name = getName();
 
       if (!arguments.length) {
@@ -449,7 +451,7 @@
     }
 
     // good
-    function() {
+    function () {
       if (!arguments.length) {
         return false;
       }
@@ -470,7 +472,7 @@
     ```javascript
     // we know this wouldn't work (assuming there
     // is no notDefined global variable)
-    function example() {
+    function example () {
       console.log(notDefined); // => throws a ReferenceError
     }
 
@@ -478,7 +480,7 @@
     // reference the variable will work due to
     // variable hoisting. Note: the assignment
     // value of `true` is not hoisted.
-    function example() {
+    function example () {
       console.log(declaredButNotAssigned); // => undefined
       var declaredButNotAssigned = true;
     }
@@ -486,7 +488,7 @@
     // The interpreter is hoisting the variable
     // declaration to the top of the scope.
     // Which means our example could be rewritten as:
-    function example() {
+    function example () {
       var declaredButNotAssigned;
       console.log(declaredButNotAssigned); // => undefined
       declaredButNotAssigned = true;
@@ -496,12 +498,12 @@
   - Anonymous function expressions hoist their variable name, but not the function assignment.
 
     ```javascript
-    function example() {
+    function example () {
       console.log(anonymous); // => undefined
 
       anonymous(); // => TypeError anonymous is not a function
 
-      var anonymous = function() {
+      var anonymous = function () {
         console.log('anonymous function expression');
       };
     }
@@ -510,26 +512,26 @@
   - Named function expressions hoist the variable name, not the function name or the function body.
 
     ```javascript
-    function example() {
+    function example () {
       console.log(named); // => undefined
 
       named(); // => TypeError named is not a function
 
       superPower(); // => ReferenceError superPower is not defined
 
-      var named = function superPower() {
+      var named = function superPower () {
         console.log('Flying');
       };
 
 
       // the same is true when the function name
       // is the same as the variable name.
-      function example() {
+      function example () {
         console.log(named); // => undefined
 
         named(); // => TypeError named is not a function
 
-        var named = function named() {
+        var named = function named () {
           console.log('named');
         };
       }
@@ -539,10 +541,10 @@
   - Function declarations hoist their name and the function body.
 
     ```javascript
-    function example() {
+    function example () {
       superPower(); // => Flying
 
-      function superPower() {
+      function superPower () {
         console.log('Flying');
       }
     }
@@ -642,7 +644,7 @@
     //
     // @param <String> tag
     // @return <Element> element
-    function make(tag) {
+    function make (tag) {
 
       // ...stuff...
 
@@ -657,7 +659,7 @@
      * @param <String> tag
      * @return <Element> element
      */
-    function make(tag) {
+    function make (tag) {
 
       // ...stuff...
 
@@ -676,7 +678,7 @@
     var active = true;
 
     // bad
-    function getType() {
+    function getType () {
       console.log('fetching type...');
       // set the default type to 'no type'
       var type = this._type || 'no type';
@@ -685,7 +687,7 @@
     }
 
     // good
-    function getType() {
+    function getType () {
       console.log('fetching type...');
 
       // set the default type to 'no type'
@@ -700,7 +702,7 @@
   - Use `// FIXME:` to annotate problems
 
     ```javascript
-    function Calculator() {
+    function Calculator () {
 
       // FIXME: shouldn't use a global here
       total = 0;
@@ -712,7 +714,7 @@
   - Use `// TODO:` to annotate solutions to problems
 
     ```javascript
-    function Calculator() {
+    function Calculator () {
 
       // TODO: total should be configurable by an options param
       this.total = 0;
@@ -730,17 +732,17 @@
 
     ```javascript
     // bad
-    function() {
+    function () {
     ∙∙∙∙var name;
     }
 
     // bad
-    function() {
+    function () {
     ∙var name;
     }
 
     // good
-    function() {
+    function () {
     ∙∙var name;
     }
     ```
@@ -748,12 +750,12 @@
 
     ```javascript
     // bad
-    function test(){
+    function test (){
       console.log('test');
     }
 
     // good
-    function test() {
+    function test () {
       console.log('test');
     }
 
@@ -769,18 +771,32 @@
       breed: 'Bernese Mountain Dog'
     });
     ```
+
+  - Place 1 space between the function name and parenthesis.
+
+    ```javascript
+    // bad
+    function test() {
+      console.log('test');
+    }
+
+    // good
+    function test () {
+      console.log('test');
+    }
+    ```
   - Place an empty newline at the end of the file.
 
     ```javascript
     // bad
-    (function(global) {
+    (function (global) {
       // ...stuff...
     })(this);
     ```
 
     ```javascript
     // good
-    (function(global) {
+    (function (global) {
       // ...stuff...
     })(this);
 
@@ -888,19 +904,19 @@
 
     ```javascript
     // bad
-    (function() {
+    (function () {
       var name = 'Skywalker'
       return name
     })()
 
     // good
-    (function() {
+    (function () {
       var name = 'Skywalker';
       return name;
     })();
 
     // good
-    ;(function() {
+    ;(function () {
       var name = 'Skywalker';
       return name;
     })();
@@ -990,12 +1006,12 @@
 
     ```javascript
     // bad
-    function q() {
+    function q () {
       // ...stuff...
     }
 
     // good
-    function query() {
+    function query () {
       // ..stuff..
     }
     ```
@@ -1007,14 +1023,14 @@
     var OBJEcttsssss = {};
     var this_is_my_object = {};
     var this-is-my-object = {};
-    function c() {};
+    function c () {};
     var u = new user({
       name: 'Bob Parr'
     });
 
     // good
     var thisIsMyObject = {};
-    function thisIsMyFunction() {};
+    function thisIsMyFunction () {};
     var user = new User({
       name: 'Bob Parr'
     });
@@ -1024,7 +1040,7 @@
 
     ```javascript
     // bad
-    function user(options) {
+    function user (options) {
       this.name = options.name;
     }
 
@@ -1033,7 +1049,7 @@
     });
 
     // good
-    function User(options) {
+    function User (options) {
       this.name = options.name;
     }
 
@@ -1053,44 +1069,36 @@
     this._firstName = 'Panda';
     ```
 
-  - When saving a reference to `this` use `_this`.
+  - When saving a reference to `this` use `self`.
 
     ```javascript
     // bad
-    function() {
-      var self = this;
-      return function() {
-        console.log(self);
-      };
-    }
-
-    // bad
-    function() {
+    function () {
       var that = this;
-      return function() {
+      return function () {
         console.log(that);
       };
     }
 
     // good
-    function() {
-      var _this = this;
-      return function() {
-        console.log(_this);
+    function () {
+      var self = this;
+      return function () {
+        console.log(self);
       };
     }
     ```
 
-  - Name your functions. This is helpful for stack traces.
+  - Name your functions for helpful stack traces.
 
     ```javascript
     // bad
-    var log = function(msg) {
+    var log = function (msg) {
       console.log(msg);
     };
 
     // good
-    var log = function log(msg) {
+    var log = function log (msg) {
       console.log(msg);
     };
     ```
@@ -1134,17 +1142,17 @@
   - It's okay to create get() and set() functions, but be consistent.
 
     ```javascript
-    function Jedi(options) {
+    function Jedi (options) {
       options || (options = {});
       var lightsaber = options.lightsaber || 'blue';
       this.set('lightsaber', lightsaber);
     }
 
-    Jedi.prototype.set = function(key, val) {
+    Jedi.prototype.set = function (key, val) {
       this[key] = val;
     };
 
-    Jedi.prototype.get = function(key) {
+    Jedi.prototype.get = function (key) {
       return this[key];
     };
     ```
@@ -1157,27 +1165,27 @@
   - Assign methods to the prototype object, instead of overwriting the prototype with a new object. Overwriting the prototype makes inheritance impossible: by resetting the prototype you'll overwrite the base!
 
     ```javascript
-    function Jedi() {
+    function Jedi () {
       console.log('new jedi');
     }
 
     // bad
     Jedi.prototype = {
-      fight: function fight() {
+      fight: function fight () {
         console.log('fighting');
       },
 
-      block: function block() {
+      block: function block () {
         console.log('blocking');
       }
     };
 
     // good
-    Jedi.prototype.fight = function fight() {
+    Jedi.prototype.fight = function fight () {
       console.log('fighting');
     };
 
-    Jedi.prototype.block = function block() {
+    Jedi.prototype.block = function block () {
       console.log('blocking');
     };
     ```
@@ -1186,12 +1194,12 @@
 
     ```javascript
     // bad
-    Jedi.prototype.jump = function() {
+    Jedi.prototype.jump = function () {
       this.jumping = true;
       return true;
     };
 
-    Jedi.prototype.setHeight = function(height) {
+    Jedi.prototype.setHeight = function (height) {
       this.height = height;
     };
 
@@ -1200,12 +1208,12 @@
     luke.setHeight(20) // => undefined
 
     // good
-    Jedi.prototype.jump = function() {
+    Jedi.prototype.jump = function () {
       this.jumping = true;
       return this;
     };
 
-    Jedi.prototype.setHeight = function(height) {
+    Jedi.prototype.setHeight = function (height) {
       this.height = height;
       return this;
     };
@@ -1220,16 +1228,16 @@
   - It's okay to write a custom toString() method, just make sure it works successfully and causes no side effects.
 
     ```javascript
-    function Jedi(options) {
+    function Jedi (options) {
       options || (options = {});
       this.name = options.name || 'no name';
     }
 
-    Jedi.prototype.getName = function getName() {
+    Jedi.prototype.getName = function getName () {
       return this.name;
     };
 
-    Jedi.prototype.toString = function toString() {
+    Jedi.prototype.toString = function toString () {
       return 'Jedi - ' + this.getName();
     };
     ```
@@ -1247,7 +1255,7 @@
 
     ...
 
-    $(this).on('listingUpdated', function(e, listingId) {
+    $(this).on('listingUpdated', function (e, listingId) {
       // do something with listingId
     });
     ```
@@ -1260,7 +1268,7 @@
 
     ...
 
-    $(this).on('listingUpdated', function(e, data) {
+    $(this).on('listingUpdated', function (e, data) {
       // do something with data.listingId
     });
     ```
@@ -1268,37 +1276,6 @@
   **[[⬆]](#TOC)**
 
 
-## <a name='modules'>Modules</a>
-
-  - The module should start with a `!`. This ensures that if a malformed module forgets to include a final semicolon there aren't errors in production when the scripts get concatenated.
-  - The file should be named with camelCase, live in a folder with the same name, and match the name of the single export.
-  - Add a method called noConflict() that sets the exported module to the previous version and returns this one.
-  - Always declare `'use strict';` at the top of the module.
-
-    ```javascript
-    // fancyInput/fancyInput.js
-
-    !function(global) {
-      'use strict';
-
-      var previousFancyInput = global.FancyInput;
-
-      function FancyInput(options) {
-        this.options = options || {};
-      }
-
-      FancyInput.noConflict = function noConflict() {
-        global.FancyInput = previousFancyInput;
-        return FancyInput;
-      };
-
-      global.FancyInput = FancyInput;
-    }(this);
-    ```
-
-    **[[⬆]](#TOC)**
-
-
 ## <a name='jquery'>jQuery</a>
 
   - Prefix jQuery object variables with a `$`.
@@ -1315,7 +1292,7 @@
 
     ```javascript
     // bad
-    function setSidebar() {
+    function setSidebar () {
       $('.sidebar').hide();
 
       // ...stuff...
@@ -1326,7 +1303,7 @@
     }
 
     // good
-    function setSidebar() {
+    function setSidebar () {
       var $sidebar = $('.sidebar');
       $sidebar.hide();
 
@@ -1376,7 +1353,7 @@
   - **Yup.**
 
     ```javascript
-    function() {
+    function () {
       return true;
     }
     ```

From 40764912746c324c195d6041e215636281448a79 Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Mon, 9 Sep 2013 21:59:12 -0500
Subject: [PATCH 2/9] remove unnecessary link

---
 README.md | 1 -
 1 file changed, 1 deletion(-)

diff --git a/README.md b/README.md
index 88bd68d740..3811e26f9a 100644
--- a/README.md
+++ b/README.md
@@ -24,7 +24,6 @@
   1. [Accessors](#accessors)
   1. [Constructors](#constructors)
   1. [Events](#events)
-  1. [Modules](#modules)
   1. [jQuery](#jquery)
   1. [ES5 Compatibility](#es5)
   1. [Testing](#testing)

From 9ce6c573d0c1ebcb55fb697f2a6c4d6f492bbf1f Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Mon, 9 Sep 2013 22:44:06 -0500
Subject: [PATCH 3/9] cleanup

---
 README.md | 16 +++-------------
 1 file changed, 3 insertions(+), 13 deletions(-)

diff --git a/README.md b/README.md
index 3811e26f9a..cc493bb82d 100644
--- a/README.md
+++ b/README.md
@@ -612,7 +612,7 @@
     if (test)
       return false;
 
-    // good
+    // okay
     if (test) return false;
 
     // good
@@ -621,10 +621,10 @@
     }
 
     // bad
-    function() { return false; }
+    function () { return false; }
 
     // good
-    function() {
+    function () {
       return false;
     }
     ```
@@ -839,16 +839,6 @@
   - Leading commas: **Nope.**
 
     ```javascript
-    // bad
-    var once
-      , upon
-      , aTime;
-
-    // good
-    var once,
-        upon,
-        aTime;
-
     // bad
     var hero = {
         firstName: 'Bob'

From a9b20f49d2cb9cdaafb7ab2847bc736c33b13212 Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Mon, 9 Sep 2013 23:48:54 -0500
Subject: [PATCH 4/9] control flow

---
 README.md | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/README.md b/README.md
index cc493bb82d..c86b356088 100644
--- a/README.md
+++ b/README.md
@@ -14,6 +14,7 @@
   1. [Variables](#variables)
   1. [Hoisting](#hoisting)
   1. [Conditional Expressions & Equality](#conditionals)
+  1. [Control Flow](#control-flow)
   1. [Blocks](#blocks)
   1. [Comments](#comments)
   1. [Whitespace](#whitespace)
@@ -602,6 +603,42 @@
 
     **[[⬆]](#TOC)**
 
+## <a name='control-flow'>Control Flow</a>
+
+  - Put `else`/`catch` on same line as previous closing `}`.
+    ```javascript
+
+    // bad
+    if (test) {
+      // stuff
+    }
+    else {
+      // stuff
+    }
+
+    // good
+    if (test) {
+      // stuff
+    } else {
+      // stuff
+    }
+
+    // bad
+    try {
+      // stuff
+    }
+    catch (e) {
+      // stuff
+    }
+
+    // good
+    try {
+      // stuff
+    } catch (e) {
+      // stuff
+    }
+
+    ```
 
 ## <a name='blocks'>Blocks</a>
 

From f058af74048d2adc4185c8f2dc6e2ddf4c148a64 Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Tue, 10 Sep 2013 00:50:20 -0500
Subject: [PATCH 5/9] remove unnecessary sections

---
 README.md | 32 --------------------------------
 1 file changed, 32 deletions(-)

diff --git a/README.md b/README.md
index c86b356088..8978f28a03 100644
--- a/README.md
+++ b/README.md
@@ -30,8 +30,6 @@
   1. [Testing](#testing)
   1. [Performance](#performance)
   1. [Resources](#resources)
-  1. [In the Wild](#in-the-wild)
-  1. [Translation](#translation)
   1. [The JavaScript Style Guide Guide](#guide-guide)
   1. [Contributors](#contributors)
   1. [License](#license)
@@ -1450,36 +1448,6 @@
 
   **[[⬆]](#TOC)**
 
-## <a name='in-the-wild'>In the Wild</a>
-
-  This is a list of organizations that are using this style guide. Send us a pull request or open an issue and we'll add you to the list.
-
-  - **Airbnb**: [airbnb/javascript](https://github.com/airbnb/javascript)
-  - **American Insitutes for Research**: [AIRAST/javascript](https://github.com/AIRAST/javascript)
-  - **Compass Learning**: [compasslearning/javascript-style-guide](https://github.com/compasslearning/javascript-style-guide)
-  - **ExactTarget**: [ExactTarget/javascript](https://github.com/ExactTarget/javascript)
-  - **GeneralElectric**: [GeneralElectric/javascript](https://github.com/GeneralElectric/javascript)
-  - **GoodData**: [gooddata/gdc-js-style](https://github.com/gooddata/gdc-js-style)
-  - **How About We**: [howaboutwe/javascript](https://github.com/howaboutwe/javascript)
-  - **MinnPost**: [MinnPost/javascript](https://github.com/MinnPost/javascript)
-  - **ModCloth**: [modcloth/javascript](https://github.com/modcloth/javascript)
-  - **National Geographic**: [natgeo/javascript](https://github.com/natgeo/javascript)
-  - **National Park Service**: [nationalparkservice/javascript](https://github.com/nationalparkservice/javascript)
-  - **Razorfish**: [razorfish/javascript-style-guide](https://github.com/razorfish/javascript-style-guide)
-  - **Shutterfly**: [shutterfly/javascript](https://github.com/shutterfly/javascript)
-  - **Userify**: [userify/javascript](https://github.com/userify/javascript)
-  - **Zillow**: [zillow/javascript](https://github.com/zillow/javascript)
-  - **ZocDoc**: [ZocDoc/javascript](https://github.com/ZocDoc/javascript)
-
-## <a name='translation'>Translation</a>
-
-  This style guide is also available in other languages:
-
-  - :de: **German**: [timofurrer/javascript-style-guide](https://github.com/timofurrer/javascript-style-guide)
-  - :jp: **Japanese**: [mitsuruog/javacript-style-guide](https://github.com/mitsuruog/javacript-style-guide)
-  - :br: **Portuguese**: [armoucar/javascript-style-guide](https://github.com/armoucar/javascript-style-guide)
-  - :cn: **Chinese**: [adamlu/javascript-style-guide](https://github.com/adamlu/javascript-style-guide)
-
 ## <a name='guide-guide'>The JavaScript Style Guide Guide</a>
 
   - [Reference](https://github.com/airbnb/javascript/wiki/The-JavaScript-Style-Guide-Guide)

From 5cb448e29600f5090b007ca4490f22e331f1786a Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Tue, 10 Sep 2013 00:58:34 -0500
Subject: [PATCH 6/9] remove cruft

---
 README.md | 206 +-----------------------------------------------------
 1 file changed, 1 insertion(+), 205 deletions(-)

diff --git a/README.md b/README.md
index 8978f28a03..ab8ea5c6dc 100644
--- a/README.md
+++ b/README.md
@@ -5,14 +5,12 @@
 
 ## <a name='TOC'>Table of Contents</a>
 
-  1. [Types](#types)
   1. [Objects](#objects)
   1. [Arrays](#arrays)
   1. [Strings](#strings)
   1. [Functions](#functions)
   1. [Properties](#properties)
   1. [Variables](#variables)
-  1. [Hoisting](#hoisting)
   1. [Conditional Expressions & Equality](#conditionals)
   1. [Control Flow](#control-flow)
   1. [Blocks](#blocks)
@@ -26,49 +24,11 @@
   1. [Constructors](#constructors)
   1. [Events](#events)
   1. [jQuery](#jquery)
-  1. [ES5 Compatibility](#es5)
-  1. [Testing](#testing)
-  1. [Performance](#performance)
   1. [Resources](#resources)
   1. [The JavaScript Style Guide Guide](#guide-guide)
   1. [Contributors](#contributors)
   1. [License](#license)
 
-## <a name='types'>Types</a>
-
-  - **Primitives**: When you access a primitive type you work directly on its value
-
-    + `string`
-    + `number`
-    + `boolean`
-    + `null`
-    + `undefined`
-
-    ```javascript
-    var foo = 1;
-    var bar = foo;
-
-    bar = 9;
-
-    console.log(foo, bar); // => 1, 9
-    ```
-  - **Complex**: When you access a complex type you work on a reference to its value
-
-    + `object`
-    + `array`
-    + `function`
-
-    ```javascript
-    var foo = [1, 2];
-    var bar = foo;
-
-    bar[0] = 9;
-
-    console.log(foo[0], bar[0]); // => 9, 9
-    ```
-
-    **[[⬆]](#TOC)**
-
 ## <a name='objects'>Objects</a>
 
   - Use the literal syntax for object creation.
@@ -377,31 +337,6 @@
     var dragonball = 'z';
     ```
 
-  - Declare unassigned variables last. This is helpful when later on you might need to assign a variable depending on one of the previous assigned variables.
-
-    ```javascript
-    // bad
-    var i;
-    var len;
-    var dragonball;
-    var items = getItems();
-    var goSportsTeam = true;
-
-    // bad
-    var i;
-    var items = getItems();
-    var dragonball;
-    var goSportsTeam = true;
-    var len;
-
-    // good
-    var items = getItems();
-    var goSportsTeam = true;
-    var dragonball;
-    var length;
-    var i;
-    ```
-
   - Assign variables at the top of their scope when not inconvenient. This helps avoid issues with variable declaration and assignment hoisting related issues.
 
     ```javascript
@@ -462,98 +397,6 @@
 
     **[[⬆]](#TOC)**
 
-
-## <a name='hoisting'>Hoisting</a>
-
-  - Variable declarations get hoisted to the top of their scope, their assignment does not.
-
-    ```javascript
-    // we know this wouldn't work (assuming there
-    // is no notDefined global variable)
-    function example () {
-      console.log(notDefined); // => throws a ReferenceError
-    }
-
-    // creating a variable declaration after you
-    // reference the variable will work due to
-    // variable hoisting. Note: the assignment
-    // value of `true` is not hoisted.
-    function example () {
-      console.log(declaredButNotAssigned); // => undefined
-      var declaredButNotAssigned = true;
-    }
-
-    // The interpreter is hoisting the variable
-    // declaration to the top of the scope.
-    // Which means our example could be rewritten as:
-    function example () {
-      var declaredButNotAssigned;
-      console.log(declaredButNotAssigned); // => undefined
-      declaredButNotAssigned = true;
-    }
-    ```
-
-  - Anonymous function expressions hoist their variable name, but not the function assignment.
-
-    ```javascript
-    function example () {
-      console.log(anonymous); // => undefined
-
-      anonymous(); // => TypeError anonymous is not a function
-
-      var anonymous = function () {
-        console.log('anonymous function expression');
-      };
-    }
-    ```
-
-  - Named function expressions hoist the variable name, not the function name or the function body.
-
-    ```javascript
-    function example () {
-      console.log(named); // => undefined
-
-      named(); // => TypeError named is not a function
-
-      superPower(); // => ReferenceError superPower is not defined
-
-      var named = function superPower () {
-        console.log('Flying');
-      };
-
-
-      // the same is true when the function name
-      // is the same as the variable name.
-      function example () {
-        console.log(named); // => undefined
-
-        named(); // => TypeError named is not a function
-
-        var named = function named () {
-          console.log('named');
-        };
-      }
-    }
-    ```
-
-  - Function declarations hoist their name and the function body.
-
-    ```javascript
-    function example () {
-      superPower(); // => Flying
-
-      function superPower () {
-        console.log('Flying');
-      }
-    }
-    ```
-
-  - For more information refer to [JavaScript Scoping & Hoisting](http://www.adequatelygood.com/2010/2/JavaScript-Scoping-and-Hoisting) by [Ben Cherry](http://www.adequatelygood.com/)
-
-    **[[⬆]](#TOC)**
-
-
-
 ## <a name='conditionals'>Conditional Expressions & Equality</a>
 
   - Use `===` and `!==` over `==` and `!=`.
@@ -994,18 +837,6 @@
     var val = parseInt(inputValue, 10);
     ```
 
-  - If for whatever reason you are doing something wild and `parseInt` is your bottleneck and need to use Bitshift for [performance reasons](http://jsperf.com/coercion-vs-casting/3), leave a comment explaining why and what you're doing.
-
-    ```javascript
-    // good
-    /**
-     * parseInt was the reason my code was slow.
-     * Bitshifting the String to coerce it to a
-     * Number made it a lot faster.
-     */
-    var val = inputValue >> 0;
-    ```
-
   - Booleans:
 
     ```javascript
@@ -1014,7 +845,7 @@
     // bad
     var hasAge = new Boolean(age);
 
-    // good
+    // okay
     var hasAge = Boolean(age);
 
     // good
@@ -1364,41 +1195,6 @@
 
     **[[⬆]](#TOC)**
 
-
-## <a name='es5'>ECMAScript 5 Compatibility</a>
-
-  - Refer to [Kangax](https://twitter.com/kangax/)'s ES5 [compatibility table](http://kangax.github.com/es5-compat-table/)
-
-  **[[⬆]](#TOC)**
-
-
-## <a name='testing'>Testing</a>
-
-  - **Yup.**
-
-    ```javascript
-    function () {
-      return true;
-    }
-    ```
-
-    **[[⬆]](#TOC)**
-
-
-## <a name='performance'>Performance</a>
-
-  - [On Layout & Web Performance](http://kellegous.com/j/2013/01/26/layout-performance/)
-  - [String vs Array Concat](http://jsperf.com/string-vs-array-concat/2)
-  - [Try/Catch Cost In a Loop](http://jsperf.com/try-catch-in-loop-cost)
-  - [Bang Function](http://jsperf.com/bang-function)
-  - [jQuery Find vs Context, Selector](http://jsperf.com/jquery-find-vs-context-sel/13)
-  - [innerHTML vs textContent for script text](http://jsperf.com/innerhtml-vs-textcontent-for-script-text)
-  - [Long String Concatenation](http://jsperf.com/ya-string-concat)
-  - Loading...
-
-  **[[⬆]](#TOC)**
-
-
 ## <a name='resources'>Resources</a>
 
 

From 1a42ff1e5d902e148f62400a289bee58f324c9c8 Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Tue, 10 Sep 2013 10:40:45 -0500
Subject: [PATCH 7/9] Update README.md

---
 README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/README.md b/README.md
index ab8ea5c6dc..004cec6f89 100644
--- a/README.md
+++ b/README.md
@@ -1,4 +1,4 @@
-# Airbnb JavaScript Style Guide() {
+# JavaScript Style Guide() {
 
 *A mostly reasonable approach to JavaScript*
 

From 408ea58a6c90a8a18f0a0e4a2f3fffd92f71fefd Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Tue, 10 Sep 2013 10:48:47 -0500
Subject: [PATCH 8/9] minor silly things

---
 README.md | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 004cec6f89..ad670fd369 100644
--- a/README.md
+++ b/README.md
@@ -1,8 +1,7 @@
-# JavaScript Style Guide() {
+# JavaScript Style Guide () {
 
 *A mostly reasonable approach to JavaScript*
 
-
 ## <a name='TOC'>Table of Contents</a>
 
   1. [Objects](#objects)

From 2c7925577e609e8fbf00fd2808b6d7eaf9c35ab9 Mon Sep 17 00:00:00 2001
From: Oliver Wong <oliver@owiber.com>
Date: Tue, 10 Sep 2013 19:58:40 -0500
Subject: [PATCH 9/9] Update README.md

---
 README.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index ad670fd369..a854681072 100644
--- a/README.md
+++ b/README.md
@@ -147,7 +147,8 @@
     var fullName = 'Bob ' + this.lastName;
     ```
 
-  - Strings longer than 80 characters should be written across multiple lines using string concatenation.
+  - Strings longer than 80 characters should be avoided and likely call for using a template system or similar.
+  - If you have a good reason to have very long strings in code, they should be written across multiple lines using string concatenation.
   - Note: If overused, long strings with concatenation could impact performance. [jsPerf](http://jsperf.com/ya-string-concat) & [Discussion](https://github.com/airbnb/javascript/issues/40)
 
     ```javascript