From f75d06bf1054a692f362239edcd1133a560c73f5 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Fri, 5 Feb 2016 20:39:45 -0800
Subject: [PATCH] tools: lint for empty character classes in regex

Enable linting rule to forbid empty character classes in regular
expressions. See http://eslint.org/docs/rules/no-empty-character-class

Organize "Possible Error" rules in .eslintrc in alphabetical order to
match eslint documentation.

PR-URL: https://github.com/nodejs/node/pull/5115
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 .eslintrc | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/.eslintrc b/.eslintrc
index 9f117dbacecb8a..98afe47e780565 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -17,6 +17,8 @@ ecmaFeatures:
 rules:
   # Possible Errors
   # list: https://github.com/eslint/eslint/tree/master/docs/rules#possible-errors
+  ## disallow control characters in regular expressions
+  no-control-regex: 2
   ## check debugger sentence
   no-debugger: 2
   ## check duplicate arguments
@@ -25,24 +27,24 @@ rules:
   no-dupe-keys: 2
   ## check duplicate switch-case
   no-duplicate-case: 2
-  ## disallow superfluous semicolons
-  no-extra-semi: 2
+  ## disallow the use of empty character classes in regular expressions
+  no-empty-character-class: 2
   ## disallow assignment of exceptional params
   no-ex-assign: 2
-  ## disallow unreachable code
-  no-unreachable: 2
-  ## require valid typeof compared string like typeof foo === 'strnig'
-  valid-typeof: 2
-  ## disallow controls characters in regular expressions
-  no-control-regex: 2
   ## disallow extra boolean casts
   no-extra-boolean-cast : 2
+  ## disallow superfluous semicolons
+  no-extra-semi: 2
   ## validate regular expressions
   no-invalid-regexp: 2
   ## forbid weird whitespace characters
   no-irregular-whitespace: 2
   ## avoid unexpected multiline expressions
   no-unexpected-multiline: 2
+  ## disallow unreachable code
+  no-unreachable: 2
+  ## require valid typeof compared string like typeof foo === 'strnig'
+  valid-typeof: 2
 
   # Best Practices
   # list: https://github.com/eslint/eslint/tree/master/docs/rules#best-practices