Reviewers: Lasse Reichstein,

Description:
Fix probably harmless thinko in StringCharAt (causes
slow-case code to be run).

Please review this at http://codereview.chromium.org/43005

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
   M     src/string.js


Index: src/string.js
===================================================================
--- src/string.js       (revision 1456)
+++ src/string.js       (working copy)
@@ -62,9 +62,9 @@

  // ECMA-262, section 15.5.4.4
  function StringCharAt(pos) {
+  var subject = ToString(this);
    var char_code = %_FastCharCodeAt(subject, index);
    if (!%_IsSmi(char_code)) {
-    var subject = ToString(this);
      var index = TO_INTEGER(pos);
      if (index >= subject.length || index < 0) return "";
      char_code = %StringCharCodeAt(subject, index);



--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to