Speed up string scanning

src/scanner.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Features shared by parsing and pre-parsing scanners.
 
 #ifndef V8_SCANNER_H_
 #define V8_SCANNER_H_
 
 #include "src/allocation.h"
@@ skipping 553 matching lines @@
     Advance();
   }
 
   // Low-level scanning support.
   template <bool capture_raw = false, bool check_surrogate = true>
   void Advance() {
     if (capture_raw) {
       AddRawLiteralChar(c0_);
     }
     c0_ = source_->Advance();
-    if (check_surrogate) HandleLeadSurrugate();
+    if (check_surrogate) HandleLeadSurrogate();
   }
 
-  void HandleLeadSurrugate() {
+  void HandleLeadSurrogate() {
     if (unibrow::Utf16::IsLeadSurrogate(c0_)) {

[marja, 2015/03/04 08:52:51]

So this function is also called when c0_ is not a lead surrogate, and that's
normal, right? I was briefly confused b/c of the name.

       uc32 c1 = source_->Advance();
       if (!unibrow::Utf16::IsTrailSurrogate(c1)) {
         source_->PushBack(c1);
       } else {
         c0_ = unibrow::Utf16::CombineSurrogatePair(c0_, c1);
       }
     }
   }
 
   void PushBack(uc32 ch) {
@@ skipping 163 matching lines @@
   bool harmony_classes_;
   // Whether we scan TEMPLATE_SPAN and TEMPLATE_TAIL
   bool harmony_templates_;
   // Whether we allow \u{xxxxx}.
   bool harmony_unicode_;
 };
 
 } }  // namespace v8::internal
 
 #endif  // V8_SCANNER_H_