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

Side by Side Diff: src/parser.cc

Issue 3382007: [Isolates] ScavengeVisitor gets member Heap*. (Closed)
Patch Set: fixed per review comments Created 10 years, 3 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/mark-compact.cc ('k') | src/runtime.cc » ('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 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 4453 matching lines...) Expand 10 before | Expand all | Expand 10 after
4464 if (has_next()) { 4464 if (has_next()) {
4465 return in()->Get(next_pos_); 4465 return in()->Get(next_pos_);
4466 } else { 4466 } else {
4467 return kEndMarker; 4467 return kEndMarker;
4468 } 4468 }
4469 } 4469 }
4470 4470
4471 4471
4472 void RegExpParser::Advance() { 4472 void RegExpParser::Advance() {
4473 if (next_pos_ < in()->length()) { 4473 if (next_pos_ < in()->length()) {
4474 StackLimitCheck check; 4474 Isolate* isolate = Isolate::Current();
4475 StackLimitCheck check(isolate);
4475 if (check.HasOverflowed()) { 4476 if (check.HasOverflowed()) {
4476 ReportError(CStrVector(Isolate::kStackOverflowMessage)); 4477 ReportError(CStrVector(Isolate::kStackOverflowMessage));
4477 } else if (ZONE->excess_allocation()) { 4478 } else if (isolate->zone()->excess_allocation()) {
4478 ReportError(CStrVector("Regular expression too large")); 4479 ReportError(CStrVector("Regular expression too large"));
4479 } else { 4480 } else {
4480 current_ = in()->Get(next_pos_); 4481 current_ = in()->Get(next_pos_);
4481 next_pos_++; 4482 next_pos_++;
4482 } 4483 }
4483 } else { 4484 } else {
4484 current_ = kEndMarker; 4485 current_ = kEndMarker;
4485 has_more_ = false; 4486 has_more_ = false;
4486 } 4487 }
4487 } 4488 }
(...skipping 844 matching lines...) Expand 10 before | Expand all | Expand 10 after
5332 parser.ParseLazy(script_source, name, 5333 parser.ParseLazy(script_source, name,
5333 start_position, end_position, is_expression); 5334 start_position, end_position, is_expression);
5334 return result; 5335 return result;
5335 } 5336 }
5336 5337
5337 5338
5338 #undef NEW 5339 #undef NEW
5339 5340
5340 5341
5341 } } // namespace v8::internal 5342 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/mark-compact.cc ('k') | src/runtime.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698