readline: use native codePointAt

Semver: patch PR-URL: #825 Reviewed-By: Chris Dickinson <[email protected]>
nodejs · Feb 23, 2015 · b41dbc2 · b41dbc2
1 parent cb22bc9
commit b41dbc2
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/lib/readline.js b/lib/readline.js
@@ -570,7 +570,7 @@ Interface.prototype._getDisplayPos = function(str) {
   var code;
   str = stripVTControlCharacters(str);
   for (var i = 0, len = str.length; i < len; i++) {
-    code = codePointAt(str, i);
+    code = str.codePointAt(i);
     if (code >= 0x10000) { // surrogates
       i++;
     }
@@ -605,7 +605,7 @@ Interface.prototype._getCursorPos = function() {
   // move the cursor to the beginning of the next line.
   if (cols + 1 === columns &&
       this.cursor < this.line.length &&
-      isFullWidthCodePoint(codePointAt(this.line, this.cursor))) {
+      isFullWidthCodePoint(this.line.codePointAt(this.cursor))) {
     rows++;
     cols = 0;
   }
@@ -1251,7 +1251,7 @@ function getStringWidth(str) {
   var width = 0;
   str = stripVTControlCharacters(str);
   for (var i = 0, len = str.length; i < len; i++) {
-    var code = codePointAt(str, i);
+    var code = str.codePointAt(i);
     if (code >= 0x10000) { // surrogates
       i++;
     }
@@ -1331,7 +1331,8 @@ function codePointAt(str, index) {
   }
   return code;
 }
-exports.codePointAt = codePointAt;
+exports.codePointAt = util.deprecate(codePointAt,
+    'codePointAt() is deprecated. Use String.prototype.codePointAt');
 
 
 /**