readline: fixes scoping bug

Signed-off-by: Fedor Indutny <fedor@indutny.com>

lib/readline.js

@@ -958,15 +958,6 @@ var escapeCodeReAnywhere = new RegExp([
 ].join('|'));
 
 function emitKeys(stream, s) {
-  var ch,
-      key = {
-        name: undefined,
-        ctrl: false,
-        meta: false,
-        shift: false
-      },
-      parts;
-
   if (util.isBuffer(s)) {
     if (s[0] > 127 && util.isUndefined(s[1])) {
       s[0] -= 128;
@@ -986,7 +977,15 @@ function emitKeys(stream, s) {
   buffer = buffer.concat(s.split(''));
 
   buffer.forEach(function(s) {
-    key.sequence = s;
+    var ch,
+        key = {
+          sequence: s,
+          name: undefined,
+          ctrl: false,
+          meta: false,
+          shift: false
+        },
+        parts;
 
     if (s === '\r') {
       // carriage return

test/simple/test-readline-interface.js

@@ -227,6 +227,7 @@ function isWarned(emitter) {
     var remainingKeypresses = keypresses.slice();
     function keypressListener (ch, key) {
       callCount++;
+      if (ch) assert(!key.code);
       assert.equal(key.sequence, remainingKeypresses.shift());
     };
     readline.emitKeypressEvents(fi);