Revert addition of no-empty rule

This also changes the places where made to support this, excluding the
changes in ReactFragment, as IMHO, it's more readable moving
`canWarnForReactFragment = false` into the catch block.

Author: bgw

.eslintrc

@@ -51,7 +51,3 @@ rules:
 
   # BROKEN. We'd like to turn these back on.
   block-scoped-var: 0 # causes a ton of noise, eslint is too picky?
-  # on-empty is broken in babel-eslint:
-  # https://github.com/babel/babel-eslint/issues/62
-  # commented blocks shouldn't be marked as empty blocks.
-  no-empty: 0

src/isomorphic/classic/element/ReactElement.js

@@ -117,7 +117,6 @@ var ReactElement = function(type, key, ref, owner, context, props) {
         writable: true
       });
     } catch (x) {
-      // IE will fail on defineProperty
     }
     this._store.validated = false;
 

src/shared/utils/Transaction.js

@@ -141,7 +141,6 @@ var Mixin = {
           try {
             this.closeAll(0);
           } catch (err) {
-            // Ignore additional errors, only throw the original
           }
         } else {
           // Since `method` didn't throw, we don't want to silence the exception
@@ -170,13 +169,12 @@ var Mixin = {
           null;
       } finally {
         if (this.wrapperInitData[i] === Transaction.OBSERVED_ERROR) {
-          // The initializer for wrapper i threw an error;
+          // The initializer for wrapper i threw an error; initialize the
+          // remaining wrappers but silence any exceptions from them to ensure
+          // that the first error is the one to bubble up.
           try {
-            // initialize the remaining wrappers,
             this.initializeAll(i + 1);
           } catch (err) {
-            // but silence any exceptions from them to ensure that the first
-            // error is the one to bubble up.
           }
         }
       }
@@ -211,13 +209,12 @@ var Mixin = {
         errorThrown = false;
       } finally {
         if (errorThrown) {
-          // The closer for wrapper i threw an error;
+          // The closer for wrapper i threw an error; close the remaining
+          // wrappers but silence any exceptions from them to ensure that the
+          // first error is the one to bubble up.
           try {
-            // close the remaining wrappers,
             this.closeAll(i + 1);
           } catch (e) {
-            // but silence any exceptions from them to ensure that the first
-            // error is the one to bubble up.
           }
         }
       }

test/lib/postDataToURL.browser.js

@@ -1,4 +1,4 @@
-/*eslint-disable brace-style, no-empty */
+/*eslint-disable brace-style*/
 function createXMLHttpRequest() {
   try {
     return new XMLHttpRequest();
@@ -13,7 +13,7 @@ function createXMLHttpRequest() {
   }
   catch (e) {}
 }
-/*eslint-enable brace-style, no-empty */
+/*eslint-enable brace-style*/
 
 function getURLSync(url) {
   var request = createXMLHttpRequest();