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

misc cleanup #5233

Closed
wants to merge 3 commits into from
Closed

misc cleanup #5233

wants to merge 3 commits into from

Commits on Feb 15, 2016

  1. src: remove unnecessary check 

    The value's type is unsigned so it will always be >= 0.
    @mscdex
    mscdex committed Feb 15, 2016
    Configuration menu
    Copy the full SHA
    4ad8e5a View commit details
    Browse the repository at this point in the history

  2. http: remove unnecessary check 

    The value passed to `process.nextTick()` which is passed to the
callback is already a valid object, so the conditional will always
evaluate to true.
    @mscdex
    mscdex committed Feb 15, 2016
    Configuration menu
    Copy the full SHA
    5875b0c View commit details
    Browse the repository at this point in the history

  3. http: remove old, confusing comment 

    The comment refers to a property (called `_pendings`) that no longer
exists.
    @mscdex
    mscdex committed Feb 15, 2016
    Configuration menu
    Copy the full SHA
    6da8293 View commit details
    Browse the repository at this point in the history