Skip to content

src: move inner classes of Environment out for readability#26824

Closed
joyeecheung wants to merge 8 commits intonodejs:masterfrom
joyeecheung:move-env-classes
Closed

src: move inner classes of Environment out for readability#26824
joyeecheung wants to merge 8 commits intonodejs:masterfrom
joyeecheung:move-env-classes

Conversation

@joyeecheung
Copy link
Member

@joyeecheung joyeecheung commented Mar 20, 2019 •
edited
Loading

This makes it easier to see what's inside Environment. I plan to group the Environment members by their purposes and comment the commonly-used ones afterwards.

Refs: #26776

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Mar 20, 2019

@joyeecheung build started: https://ci.nodejs.org/blue/organizations/jenkins/node-test-pull-request-lite-pipeline/detail/node-test-pull-request-lite-pipeline/2991/pipeline

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. labels Mar 20, 2019
@joyeecheung
Copy link
Member Author

joyeecheung commented Mar 20, 2019

CI: https://ci.nodejs.org/job/node-test-pull-request/21693/

gengjiawen
gengjiawen reviewed Mar 21, 2019
View reviewed changes
src/env.h

class ShouldNotAbortOnUncaughtScope {
public:
explicit inline ShouldNotAbortOnUncaughtScope(Environment* env);
Copy link
Member

@gengjiawen gengjiawen Mar 21, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like forget to format this class ?

Copy link
Member Author

@joyeecheung joyeecheung Mar 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, fixed

@joyeecheung
Copy link
Member Author

joyeecheung commented Mar 22, 2019

CI: https://ci.nodejs.org/job/node-test-pull-request/21782/

@joyeecheung joyeecheung mentioned this pull request Mar 22, 2019
Closed
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Mar 25, 2019

CI: https://ci.nodejs.org/job/node-test-pull-request/21868/

@BridgeAR BridgeAR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 25, 2019
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Mar 25, 2019

CI: https://ci.nodejs.org/job/node-test-pull-request/21881/

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Mar 26, 2019

CI: https://ci.nodejs.org/job/node-test-pull-request/21933/

@joyeecheung
Copy link
Member Author

joyeecheung commented Mar 27, 2019

Landed in a8eac78...9d854fb

joyeecheung added a commit that referenced this pull request Mar 27, 2019
@joyeecheung
src: move AsyncHooks out of Environment
5197053 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
joyeecheung added a commit that referenced this pull request Mar 27, 2019
@joyeecheung
src: move AsyncCallbackScope out of Environment
cc7bb85 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
joyeecheung added a commit that referenced this pull request Mar 27, 2019
@joyeecheung
src: move ImmediateInfo out of Environment
4565698 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
joyeecheung added a commit that referenced this pull request Mar 27, 2019
@joyeecheung
src: move TickInfo out of Environment
2424661 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
joyeecheung added a commit that referenced this pull request Mar 27, 2019
@joyeecheung
src: move TrackingTraceStateObserver out of Environment
9f37d3c 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
joyeecheung added a commit that referenced this pull request Mar 27, 2019
@joyeecheung
src: move ShouldNotAbortOnUncaughtScope out of Environment
9d854fb 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit to targos/node that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move AsyncHooks out of Environment
16f90ef 
PR-URL: nodejs#26824
Refs: nodejs#26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit to targos/node that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move AsyncCallbackScope out of Environment
0fa0fe0 
PR-URL: nodejs#26824
Refs: nodejs#26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit to targos/node that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move ImmediateInfo out of Environment
4999a89 
PR-URL: nodejs#26824
Refs: nodejs#26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit to targos/node that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move TickInfo out of Environment
9d600a3 
PR-URL: nodejs#26824
Refs: nodejs#26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit to targos/node that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move TrackingTraceStateObserver out of Environment
2e2efdd 
PR-URL: nodejs#26824
Refs: nodejs#26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit to targos/node that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move ShouldNotAbortOnUncaughtScope out of Environment
41db3e4 
PR-URL: nodejs#26824
Refs: nodejs#26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move AsyncHooks out of Environment
4af9ff0 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move AsyncCallbackScope out of Environment
6de1220 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move ImmediateInfo out of Environment
495e5e9 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move TickInfo out of Environment
bc69a81 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move TrackingTraceStateObserver out of Environment
7e7f077 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Mar 27, 2019
@joyeecheung @targos
src: move ShouldNotAbortOnUncaughtScope out of Environment
f50c9c6 
PR-URL: #26824
Refs: #26776
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos targos mentioned this pull request Mar 27, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gengjiawen gengjiawen gengjiawen left review comments

@jasnell jasnell jasnell approved these changes

@addaleax addaleax addaleax approved these changes

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@joyeecheung @nodejs-github-bot @jasnell @addaleax @gengjiawen @BridgeAR