Upgrade ESLint to 2.9, add new rule definitions

Author: robertrossmann

Diff for: package.json

@@ -17,7 +17,7 @@
   "license": "BSD-3-Clause",
   "main": "default",
   "peerDependencies": {
-    "eslint": "^2.8.0"
+    "eslint": "^2.9.0"
   },
   "repository": {
     "type": "git",
@@ -27,6 +27,6 @@
     "lint": "eslint standard environments coding-styles unused.js default.js"
   },
   "devDependencies": {
-    "eslint": "^2.8.0"
+    "eslint": "^2.9.0"
   }
 }

Diff for: standard/best-practices.js

@@ -191,6 +191,10 @@ module.exports = {
     // that can be replaced with the normal function invocation.
     'no-useless-call': 2,
 
+    // Disallow unnecessary computed property keys on objects
+    // It’s unnecessary to use computed properties when the property name is a literal.
+    'no-useless-computed-key': 1,
+
     // Disallow unncessary concatenation of strings
     // This rule aims to flag the concatenation of 2 literals when they could be combined into a
     // single literal.

Diff for: standard/known-errors.js

@@ -133,6 +133,13 @@ module.exports = {
     // block exist after a `return`, `throw`, `break`, or `continue` statement.
     'no-unreachable': 2,
 
+    // Disallow control flow statements in finally blocks
+    // JavaScript suspends the control flow statements of `try` and `catch` blocks until the
+    // execution of `finally` block finishes. So, when `return`, `throw`, `break`, or `continue` is
+    // used in `finally`, control flow statements inside `try` and `catch` are overwritten, which is
+    // considered as unexpected behavior.
+    'no-unsafe-finally': 2,
+
     // Disallow Unused Labels
     // Labels that are declared and not used anywhere in the code are most likely an error due to
     // incomplete refactoring.