Skip to content

src: fix MallocedBuffer move assignment operator - #20883

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:fix-malloc-move-assign
Closed

src: fix MallocedBuffer move assignment operator#20883
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:fix-malloc-move-assign

Conversation

@addaleax

Copy link
Copy Markdown
Member

Split out from #20876

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

@addaleaxaddaleax added c++ Issues and PRs that require attention from people who are familiar with C++. fast-track PRs that do not need to wait for 48 hours to land. labels May 22, 2018
@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label May 22, 2018
@addaleax

Copy link
Copy Markdown
MemberAuthor

CI: https://ci.nodejs.org/job/node-test-commit/18669/

Feel free to 👍 this comment if you think fast-tracking makes sense.

@apapirovski

Copy link
Copy Markdown
Contributor

Last CI was troubled by infra problems: https://ci.nodejs.org/job/node-test-pull-request/15044/

@apapirovskiapapirovski added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 23, 2018
@BridgeAR

Copy link
Copy Markdown
Member

Landed in cfc9d04 🎉

BridgeAR pushed a commit to BridgeAR/node that referenced this pull request May 25, 2018
PR-URL: nodejs#20883
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
targos pushed a commit that referenced this pull request May 25, 2018
PR-URL: #20883
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@MylesBorinsMylesBorins mentioned this pull request May 29, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs 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++.fast-trackPRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

@addaleax@apapirovski@BridgeAR@jasnell@benjamingr@cjihrig@tniessen@joyeecheung@nodejs-github-bot