Skip to content
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

src: make AsyncWrap constructors delegate #19366

Closed
wants to merge 1 commit into from
Closed

src: make AsyncWrap constructors delegate #19366

wants to merge 1 commit into from

Commits on Mar 15, 2018

  1. src: make AsyncWrap constructors delegate 

    Currently, there is an AsyncWrap constructor that is only used by
PromiseWrap. This constructor has a body which is very similar
to the other AsyncWrap constructor.

This commit suggests updating the private constructor that is used
by PromiseWrap and also have the second constructor delegate to this
one to avoid the code duplication.
    @danbev
    danbev committed Mar 15, 2018
    Configuration menu
    Copy the full SHA
    893b461 View commit details
    Browse the repository at this point in the history