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

http2: minor cleanup of compat, tests #15254

Closed
wants to merge 1 commit into from
Closed

http2: minor cleanup of compat, tests #15254

wants to merge 1 commit into from

Commits on Sep 12, 2017

  1. http2: cleanup of h2 compat layer, add tests 

    Remove unnecessary variable assignments, remove unreachable code
pathways, remove path getter and setter, and other very minor
cleanup. Only remove reference to stream on nextTick so that
users and libraries can access it in the finish event.

Fixes: nodejs#15313
Refs: expressjs/express#3390
    @apapirovski
    apapirovski committed Sep 12, 2017
    Configuration menu
    Copy the full SHA
    57ab2dc View commit details
    Browse the repository at this point in the history