Closed hzuo closed 1 year ago
Base: 83.61% // Head: 83.61% // Increases project coverage by +0.00%
:tada:
Coverage data is based on head (
544041e
) compared to base (ca7f36f
). Patch has no changes to coverable lines.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
The WASM special-casing for
ahash
introduced by #1297 is problematic for a couple of reasons:compile-time-rng
feature is useless without first disablingruntime-rng
(which is enabled as a default feature). When both are enabled,runtime-rng
takes precendence but we end up bringing in extraneous dependencies forcompile-time-rng
. It's also misleading to see "compile-time-rng" when in reality it's still runtime-rng.compile-time-rng
(which we could by settingdefault-features = false
), there's no reason to includegetrandom
at all since there's no need to get randomness at runtime. But IMO it's a bad default since it leads to non-deterministic / uncacheable builds - see https://github.com/rust-lang/hashbrown/pull/155.target_arch
also made this config apply towasm32-unknown-emscripten
builds too, even though no special-casing is required for Emscripten. See getrandom's docs on WASM support.