Warn in case of using the name option with hiredis set

Author: Ruben Bridgewater

benchmark/index.js

@@ -65,8 +65,7 @@ var parserHiRedis = new Parser({
 var parser = new Parser({
   returnReply: checkReply,
   returnError: returnError,
-  returnFatalError: returnError,
-  name: 'javascript'
+  returnFatalError: returnError
 })
 
 // BULK STRINGS

benchmark/old/parser.js

@@ -1,12 +1,12 @@
 'use strict'
 
 var parsers = {
-  Javascript: require('./javascript')
+  javascript: require('./javascript')
 }
 
 // Hiredis might not be installed
 try {
-  parsers.Hiredis = require('./hiredis')
+  parsers.hiredis = require('./hiredis')
 } catch (err) { /* ignore errors */ }
 
 function Parser (options) {
@@ -45,9 +45,9 @@ function Parser (options) {
   }
 
   if (options.name === 'javascript' || !parsers.hiredis || options.stringNumbers) {
-    parser = new parsers.Javascript(innerOptions)
+    parser = new parsers.javascript(innerOptions) // eslint-disable-line new-cap
   } else {
-    parser = new parsers.Hiredis(innerOptions)
+    parser = new parsers.hiredis(innerOptions) // eslint-disable-line new-cap
   }
 
   parser.returnError = options.returnError

lib/parser.js

@@ -214,6 +214,7 @@ function JavascriptRedisParser (options) {
   if (options.name === 'hiredis') {
     try {
       var Hiredis = require('../test/hiredis')
+      console.error(new TypeError('Using the hiredis parser is discouraged. Please remove the name option.').stack)
       return new Hiredis(options)
     } catch (e) {}
   }