Skip to content

(v7.x backport) net: remove an unused internal module assertPort - #12850

Closed
watilde wants to merge 1 commit into
nodejs:v7.x-stagingfrom
watilde:backport-11812
Closed

(v7.x backport) net: remove an unused internal module assertPort#12850
watilde wants to merge 1 commit into
nodejs:v7.x-stagingfrom
watilde:backport-11812

Conversation

@watilde

Copy link
Copy Markdown
Member

Backport of #11812 to 7.x:

net: remove an unused internal module assertPort

A module assertPort in lib/internal/net.js is not used anymore.

Checklist
Affected core subsystem(s)

net

A module `assertPort` in `lib/internal/net.js` is not used anymore.
Refs: nodejs#11667
PR-URL: nodejs#11812
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
@nodejs-github-botnodejs-github-bot added net Issues and PRs related to the net subsystem. v7.x labels May 5, 2017
@watilde

Copy link
Copy Markdown
MemberAuthor

@watilde

Copy link
Copy Markdown
MemberAuthor

Landed in 33ea7a2

@watildewatilde closed this May 12, 2017
@watilde
watilde deleted the backport-11812 branch May 12, 2017 08:23
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

netIssues and PRs related to the net subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@watilde@benjamingr@nodejs-github-bot