Skip to content

src: reduce macro usage #29357

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
Closed
addaleax wants to merge 2 commits into from
Closed

src: reduce macro usage #29357

addaleax wants to merge 2 commits into from

Conversation

@addaleax
Copy link
Member

@addaleax addaleax commented Aug 28, 2019

src: inline SLICE_START_END() in node_buffer.cc

This macro is only used once, so it doesn’t need to be a macro.

src: turn GET_OFFSET() into an inline function

There’s no need for this to be a macro.

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

@addaleax
src: inline SLICE_START_END() in node_buffer.cc
80c24c7 
This macro is only used once, so it doesn’t need to be a macro.
@addaleax
src: turn GET_OFFSET() into an inline function
b00f2db 
There’s no need for this to be a macro.
@nodejs-github-bot nodejs-github-bot added buffer Issues and PRs related to the buffer subsystem. c++ Issues and PRs that require attention from people who are familiar with C++. fs Issues and PRs related to the fs subsystem / file system. labels Aug 28, 2019
@addaleax addaleax added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. lib / src Issues and PRs related to general changes in the lib or src directory. and removed buffer Issues and PRs related to the buffer subsystem. fs Issues and PRs related to the fs subsystem / file system. labels Aug 28, 2019
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Aug 28, 2019

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

mhdawson
mhdawson approved these changes Aug 29, 2019
View reviewed changes
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

danbev pushed a commit that referenced this pull request Sep 2, 2019
@addaleax @danbev
src: inline SLICE_START_END() in node_buffer.cc
5a52bda 
This macro is only used once, so it doesn’t need to be a macro.

PR-URL: #29357
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Masashi Hirano <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
danbev pushed a commit that referenced this pull request Sep 2, 2019
@addaleax @danbev
src: turn GET_OFFSET() into an inline function
0f3aa81 
There’s no need for this to be a macro.

PR-URL: #29357
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Masashi Hirano <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@danbev
Copy link
Contributor

danbev commented Sep 2, 2019

Landed in 5a52bda, and 0f3aa81.

@danbev danbev closed this Sep 2, 2019
BridgeAR pushed a commit that referenced this pull request Sep 3, 2019
@addaleax @BridgeAR
src: inline SLICE_START_END() in node_buffer.cc
00ed7d1 
This macro is only used once, so it doesn’t need to be a macro.

PR-URL: #29357
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Masashi Hirano <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
BridgeAR pushed a commit that referenced this pull request Sep 3, 2019
@addaleax @BridgeAR
src: turn GET_OFFSET() into an inline function
fd7c2be 
There’s no need for this to be a macro.

PR-URL: #29357
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Masashi Hirano <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@BridgeAR BridgeAR mentioned this pull request Sep 3, 2019
Merged
BridgeAR pushed a commit that referenced this pull request Sep 4, 2019
@addaleax @BridgeAR
src: inline SLICE_START_END() in node_buffer.cc
2666e00 
This macro is only used once, so it doesn’t need to be a macro.

PR-URL: #29357
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Masashi Hirano <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
BridgeAR pushed a commit that referenced this pull request Sep 4, 2019
@addaleax @BridgeAR
src: turn GET_OFFSET() into an inline function
b964bdd 
There’s no need for this to be a macro.

PR-URL: #29357
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Masashi Hirano <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@lpinca lpinca lpinca approved these changes

@cjihrig cjihrig cjihrig approved these changes

@gengjiawen gengjiawen gengjiawen approved these changes

@richardlau richardlau richardlau approved these changes

@mhdawson mhdawson mhdawson approved these changes

+2 more reviewers

@patilharshal16 patilharshal16 patilharshal16 approved these changes

@shisama shisama shisama approved these changes

Reviewers whose approvals may not affect merge requirements

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.