Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(161)

Side by Side Diff: src/json-parser.h

Issue 1731063007: Remove the global Strength enum class completely. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@local_strong-remove-literals
Patch Set: Remove more cruft. Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/js/v8natives.js ('k') | src/messages.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef V8_JSON_PARSER_H_ 5 #ifndef V8_JSON_PARSER_H_
6 #define V8_JSON_PARSER_H_ 6 #define V8_JSON_PARSER_H_
7 7
8 #include "src/char-predicates.h" 8 #include "src/char-predicates.h"
9 #include "src/conversions.h" 9 #include "src/conversions.h"
10 #include "src/debug/debug.h" 10 #include "src/debug/debug.h"
(...skipping 518 matching lines...) Expand 10 before | Expand all | Expand 10 after
529 } 529 }
530 } 530 }
531 AdvanceSkipWhitespace(); 531 AdvanceSkipWhitespace();
532 // Allocate a fixed array with all the elements. 532 // Allocate a fixed array with all the elements.
533 Handle<FixedArray> fast_elements = 533 Handle<FixedArray> fast_elements =
534 factory()->NewFixedArray(elements.length(), pretenure_); 534 factory()->NewFixedArray(elements.length(), pretenure_);
535 for (int i = 0, n = elements.length(); i < n; i++) { 535 for (int i = 0, n = elements.length(); i < n; i++) {
536 fast_elements->set(i, *elements[i]); 536 fast_elements->set(i, *elements[i]);
537 } 537 }
538 Handle<Object> json_array = factory()->NewJSArrayWithElements( 538 Handle<Object> json_array = factory()->NewJSArrayWithElements(
539 fast_elements, FAST_ELEMENTS, Strength::WEAK, pretenure_); 539 fast_elements, FAST_ELEMENTS, pretenure_);
540 return scope.CloseAndEscape(json_array); 540 return scope.CloseAndEscape(json_array);
541 } 541 }
542 542
543 543
544 template <bool seq_one_byte> 544 template <bool seq_one_byte>
545 Handle<Object> JsonParser<seq_one_byte>::ParseJsonNumber() { 545 Handle<Object> JsonParser<seq_one_byte>::ParseJsonNumber() {
546 bool negative = false; 546 bool negative = false;
547 int beg_pos = position_; 547 int beg_pos = position_;
548 if (c0_ == '-') { 548 if (c0_ == '-') {
549 Advance(); 549 Advance();
(...skipping 287 matching lines...) Expand 10 before | Expand all | Expand 10 after
837 DCHECK_EQ('"', c0_); 837 DCHECK_EQ('"', c0_);
838 // Advance past the last '"'. 838 // Advance past the last '"'.
839 AdvanceSkipWhitespace(); 839 AdvanceSkipWhitespace();
840 return result; 840 return result;
841 } 841 }
842 842
843 } // namespace internal 843 } // namespace internal
844 } // namespace v8 844 } // namespace v8
845 845
846 #endif // V8_JSON_PARSER_H_ 846 #endif // V8_JSON_PARSER_H_
OLDNEW
« no previous file with comments | « src/js/v8natives.js ('k') | src/messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698