Improve ScanJsonNumber.

src/scanner.h

 // Copyright 2010 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 130 matching lines @@
 
 class JsonScanner : public Scanner {
  public:
   JsonScanner();
 
   void Initialize(UC16CharacterStream* source);
 
   // Returns the next token.
   Token::Value Next();
 
+  double number;

[Lasse Reichstein, 2011/02/04 13:53:36]

Make it a private (or protected) property and call it number_, then make a
function called number() that returns it.
This way, the number field is publicly writable.

[sandholm, 2011/02/04 14:26:37]

Done.

+
  protected:
   // Skip past JSON whitespace (only space, tab, newline and carrige-return).
   bool SkipJsonWhiteSpace();
 
   // Scan a single JSON token. The JSON lexical grammar is specified in the
   // ECMAScript 5 standard, section 15.12.1.1.
   // Recognizes all of the single-character tokens directly, or calls a function
   // to scan a number, string or identifier literal.
   // The only allowed whitespace characters between tokens are tab,
   // carriage-return, newline and space.
@@ skipping 15 matching lines @@
 
   // Used to recognizes one of the literals "true", "false", or "null". These
   // are the only valid JSON identifiers (productions JSONBooleanLiteral,
   // JSONNullLiteral).
   Token::Value ScanJsonIdentifier(const char* text, Token::Value token);
 };
 
 } }  // namespace v8::internal
 
 #endif  // V8_SCANNER_H_