diff options
| author | Matt Mastracci <matthew@mastracci.com> | 2024-03-07 10:00:43 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-03-07 10:00:43 -0700 |
| commit | 4791d16a8efc42fb40ffab79bcdae4f0e106cd89 (patch) | |
| tree | 3d91c75b0d822ba68797da8a33091597f99f7af6 /cli/npm | |
| parent | 594d8397ad46a90389bec9a76afde1bc7f1fa35b (diff) | |
perf(cli): use faster_hex (#22761)
`cli::util::checksum` was showing up on flame graphs because it was
concatenating allocated strings. We can use `faster-hex` to improve it.
Diffstat (limited to 'cli/npm')
| -rw-r--r-- | cli/npm/managed/tarball.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/npm/managed/tarball.rs b/cli/npm/managed/tarball.rs index 90cae0cd2..1267b13d8 100644 --- a/cli/npm/managed/tarball.rs +++ b/cli/npm/managed/tarball.rs @@ -61,7 +61,7 @@ fn verify_tarball_integrity( let mut hash_ctx = Context::new(&ring::digest::SHA1_FOR_LEGACY_USE_ONLY); hash_ctx.update(data); let digest = hash_ctx.finish(); - let tarball_checksum = hex::encode(digest.as_ref()); + let tarball_checksum = faster_hex::hex_string(digest.as_ref()); (tarball_checksum, hex) } NpmPackageVersionDistInfoIntegrity::UnknownIntegrity(integrity) => { |