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

Side by Side Diff: runtime/vm/report.cc

Issue 1299653003: Migrate LongJumpScope to Thread. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Assert current isolate. Created 5 years, 4 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 | « runtime/vm/precompiler.cc ('k') | runtime/vm/snapshot.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 (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "vm/report.h" 5 #include "vm/report.h"
6 6
7 #include "vm/code_patcher.h" 7 #include "vm/code_patcher.h"
8 #include "vm/exceptions.h" 8 #include "vm/exceptions.h"
9 #include "vm/flags.h" 9 #include "vm/flags.h"
10 #include "vm/longjump.h" 10 #include "vm/longjump.h"
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 // Script is unknown. 81 // Script is unknown.
82 // Append the formatted error or warning message. 82 // Append the formatted error or warning message.
83 result = String::NewFormatted("%s: ", message_header); 83 result = String::NewFormatted("%s: ", message_header);
84 result = String::Concat(result, message); 84 result = String::Concat(result, message);
85 } 85 }
86 return result.raw(); 86 return result.raw();
87 } 87 }
88 88
89 89
90 void Report::LongJump(const Error& error) { 90 void Report::LongJump(const Error& error) {
91 Isolate::Current()->long_jump_base()->Jump(1, error); 91 Thread::Current()->long_jump_base()->Jump(1, error);
92 UNREACHABLE(); 92 UNREACHABLE();
93 } 93 }
94 94
95 95
96 void Report::LongJumpF(const Error& prev_error, 96 void Report::LongJumpF(const Error& prev_error,
97 const Script& script, intptr_t token_pos, 97 const Script& script, intptr_t token_pos,
98 const char* format, ...) { 98 const char* format, ...) {
99 va_list args; 99 va_list args;
100 va_start(args, format); 100 va_start(args, format);
101 LongJumpV(prev_error, script, token_pos, format, args); 101 LongJumpV(prev_error, script, token_pos, format, args);
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
240 trace_warning.AddProperty("type", "JSCompatibilityWarning"); 240 trace_warning.AddProperty("type", "JSCompatibilityWarning");
241 trace_warning.AddProperty("script", script); 241 trace_warning.AddProperty("script", script);
242 trace_warning.AddProperty("tokenPos", token_pos); 242 trace_warning.AddProperty("tokenPos", token_pos);
243 trace_warning.AddProperty("message", message); 243 trace_warning.AddProperty("message", message);
244 } 244 }
245 trace_buffer->Trace(micros, js.ToCString(), true); // Already escaped. 245 trace_buffer->Trace(micros, js.ToCString(), true); // Already escaped.
246 } 246 }
247 247
248 } // namespace dart 248 } // namespace dart
249 249
OLDNEW
« no previous file with comments | « runtime/vm/precompiler.cc ('k') | runtime/vm/snapshot.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698