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.

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

Remove sip::Hasher::short_write. #69471

Merged
merged 1 commit into from
Mar 15, 2020

Commits on Feb 25, 2020

  1. Remove sip::Hasher::short_write.

    `sip::Hasher::short_write` is currently unused. It is called by
    `sip::Hasher::write_{u8,usize}`, but those methods are also unused,
    because `DefaultHasher`, `SipHasher` and `SipHasher13` don't implement
    any of the `write_xyz` methods, so all their write operations end up
    calling `sip::Hasher::write`.
    
    (I confirmed this by inserting a `panic!` in `sip::Hasher::short_write`
    and running the tests -- they all passed.)
    
    The alternative would be to add all the missing `write_xyz` methods.
    This does give some significant speed-ups, but it hurts compile times a
    little in some cases. See rust-lang#69152 for details. This commit does the
    conservative thing and doesn't change existing behaviour.
    nnethercote committed Feb 25, 2020
    Configuration menu
    Copy the full SHA
    54d1c50 View commit details
    Browse the repository at this point in the history