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: suppress coverity message #7587

Merged
merged 1 commit into from
Jul 8, 2016
Merged

src: suppress coverity message #7587

merged 1 commit into from
Jul 8, 2016

Commits on Jul 8, 2016

  1. src: suppress coverity message 

    Coverity marked a change in 630096b as a constant expression.
However, on platforms where sizeof(int64_t) > sizeof(size_t),
this should not be the case. This commit flags the comparison
as OK to coverity.

PR-URL: nodejs#7587
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
    @cjihrig
    cjihrig committed Jul 8, 2016
    Configuration menu
    Copy the full SHA
    18ae74c View commit details
    Browse the repository at this point in the history