Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor(core): decode JsStackFrames using serde_v8 #9902

Merged
merged 5 commits into from
Mar 27, 2021
Merged

refactor(core): decode JsStackFrames using serde_v8 #9902

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ddd7a86
core/error: decode JsStackFrames using serde_v8
AaronO Mar 26, 2021
10505a3
JsStackFrame: use serde camelCase
AaronO Mar 26, 2021
8352f3b
Use serde rename to fix inconsistent case of JsStackFrame's isToplevel
AaronO Mar 26, 2021
268cc75
restore filter of non-array values for error.__callSiteEvals
AaronO Mar 26, 2021
ad8915a
core: handle NativeJsError.stack as before since it's "special"
AaronO Mar 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use serde rename to fix inconsistent case of JsStackFrame's isToplevel
  • Loading branch information
@AaronO
AaronO committed Mar 26, 2021
commit 8352f3bd54b2d085ccdf4017dc876eabf12814d0
3 changes: 3 additions & 0 deletions core/error.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,9 @@ pub struct JsStackFrame {
pub line_number: Option<i64>,
pub column_number: Option<i64>,
pub eval_origin: Option<String>,
// Warning! isToplevel has inconsistent snake<>camel case, "typo" originates in v8:
// https://source.chromium.org/search?q=isToplevel&sq=&ss=chromium%2Fchromium%2Fsrc:v8%2F
#[serde(rename = "isToplevel")]
pub is_top_level: Option<bool>,
pub is_eval: bool,
pub is_native: bool,
Expand Down