Home Original page

src: move DeleteFnPtr into util.h by addaleax · Pull Request #20885 · nodejs/node

@addaleax 

This is more generally useful than just in a crypto context.

@addaleax 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 72 hours to land.

labels

May 22, 2018

@nodejs-github-bot nodejs-github-bot added c++

Issues and PRs that require attention from people who are familiar with C++.

crypto

Issues and PRs related to the crypto subsystem.

labels

May 22, 2018

@addaleax addaleax added lib / src

Issues and PRs related to general changes in the lib or src directory.

and removed crypto

Issues and PRs related to the crypto subsystem.

labels

May 22, 2018

benjamingr

joyeecheung

@apapirovski apapirovski added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

May 23, 2018

tniessen

tniessen pushed a commit that referenced this pull request

May 25, 2018 
This is more generally useful than just in a crypto context.

PR-URL: #20885
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.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 
This is more generally useful than just in a crypto context.

PR-URL: #20885
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.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>