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

v8 inspect and ScopeInfo/Context objects #211

Closed
mmarchini opened this issue Jul 4, 2018 · 0 comments
Closed

v8 inspect and ScopeInfo/Context objects #211

mmarchini opened this issue Jul 4, 2018 · 0 comments
Labels
enhancement feature request

Comments

@mmarchini
Copy link
Contributor

v8 inspect don't work with ScopeInfo and Context objects, and it probably should.
@mmarchini mmarchini mentioned this issue Jul 4, 2018
Closed
kisg pushed a commit to paul99/v8mips that referenced this issue Jul 28, 2018
[postmortem] add ScopeInfo and Context types
09bca09 
The metadata introduced in this patch will be useful for postmortem
tools to inspect Contexts and ScopeInfos (see
nodejs/llnode#211).

R=bmeurer@google.com, yangguo@google.com

Change-Id: I927fcab4014d128bd782046c1ecb9ee045723e95
Reviewed-on: https://chromium-review.googlesource.com/1153858
Reviewed-by: Yang Guo <yangguo@chromium.org>
Commit-Queue: Yang Guo <yangguo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#54768}
mmarchini pushed a commit to mmarchini/node that referenced this issue Aug 1, 2018
deps: cherry-pick 09bca09 from upstream V8
b631495 
Original commit message:

    [postmortem] add ScopeInfo and Context types

    The metadata introduced in this patch will be useful for postmortem
    tools to inspect Contexts and ScopeInfos (see
    nodejs/llnode#211).

    R=bmeurer@google.com, yangguo@google.com

    Change-Id: I927fcab4014d128bd782046c1ecb9ee045723e95
    Reviewed-on: https://chromium-review.googlesource.com/1153858
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Commit-Queue: Yang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{nodejs#54768}

Refs: v8/v8@09bca09
mmarchini pushed a commit to nodejs/node that referenced this issue Aug 6, 2018
deps: cherry-pick 09bca09 from upstream V8
a4c1cf5 
Original commit message:

    [postmortem] add ScopeInfo and Context types

    The metadata introduced in this patch will be useful for postmortem
    tools to inspect Contexts and ScopeInfos (see
    nodejs/llnode#211).

    R=bmeurer@google.com, yangguo@google.com

    Change-Id: I927fcab4014d128bd782046c1ecb9ee045723e95
    Reviewed-on: https://chromium-review.googlesource.com/1153858
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Commit-Queue: Yang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#54768}

Refs: v8/v8@09bca09

PR-URL: #22068
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
targos pushed a commit to nodejs/node that referenced this issue Aug 11, 2018
@targos
deps: cherry-pick 09bca09 from upstream V8
e1dcb79 
Original commit message:

    [postmortem] add ScopeInfo and Context types

    The metadata introduced in this patch will be useful for postmortem
    tools to inspect Contexts and ScopeInfos (see
    nodejs/llnode#211).

    R=bmeurer@google.com, yangguo@google.com

    Change-Id: I927fcab4014d128bd782046c1ecb9ee045723e95
    Reviewed-on: https://chromium-review.googlesource.com/1153858
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Commit-Queue: Yang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#54768}

Refs: v8/v8@09bca09

PR-URL: #22068
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
mmarchini pushed a commit to mmarchini/llnode that referenced this issue Aug 20, 2018
src: inspect context objects (Node.js v10.x+)
8676013 
This patch teaches llnode how to inspect Context objects. This is useful
to inspect context variables.

Fix: nodejs#211
hyj1991 pushed a commit to aliyun-node/llnode that referenced this issue Sep 3, 2018
src: inspect context objects (Node.js v10.x+)
9174ca2 
This patch teaches llnode how to inspect Context objects. This is useful
to inspect context variables.

PR-URL: nodejs#201
Fixes: nodejs#211
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Drieger pushed a commit to Drieger/llnode that referenced this issue Sep 4, 2018
@Drieger
src: inspect context objects (Node.js v10.x+)
2c72daa 
This patch teaches llnode how to inspect Context objects. This is useful
to inspect context variables.

PR-URL: nodejs#201
Fixes: nodejs#211
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
targos pushed a commit to targos/node that referenced this issue Sep 7, 2018
@targos
deps: cherry-pick 09bca09 from upstream V8
30b0796 
Original commit message:

    [postmortem] add ScopeInfo and Context types

    The metadata introduced in this patch will be useful for postmortem
    tools to inspect Contexts and ScopeInfos (see
    nodejs/llnode#211).

    R=bmeurer@google.com, yangguo@google.com

    Change-Id: I927fcab4014d128bd782046c1ecb9ee045723e95
    Reviewed-on: https://chromium-review.googlesource.com/1153858
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Commit-Queue: Yang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{nodejs#54768}

Refs: v8/v8@09bca09

PR-URL: nodejs#22068
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
targos pushed a commit to targos/node that referenced this issue Sep 7, 2018
@targos
deps: cherry-pick 09bca09 from upstream V8
127e703 
Original commit message:

    [postmortem] add ScopeInfo and Context types

    The metadata introduced in this patch will be useful for postmortem
    tools to inspect Contexts and ScopeInfos (see
    nodejs/llnode#211).

    R=bmeurer@google.com, yangguo@google.com

    Change-Id: I927fcab4014d128bd782046c1ecb9ee045723e95
    Reviewed-on: https://chromium-review.googlesource.com/1153858
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Commit-Queue: Yang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{nodejs#54768}

Refs: v8/v8@09bca09

PR-URL: nodejs#22068
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mmarchini