maurice/snix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(tvix/glue/builtins/import): Drop useless bstring conversion

Browse source 
In a previous refactoring CL this into_bstring method was accidentally
kept, when we don't need it and can just to_str directly.

Change-Id: Idd531d508b8fd530611b213d0164e7aaf0e87d80
Reviewed-on: https://cl.tvl.fyi/c/depot/+/12631
Tested-by: BuildkiteCI
Reviewed-by: flokli <flokli@flokli.de>
Autosubmit: Ilan Joselevich <personal@ilanjoselevich.com>
This commit is contained in:
Ilan Joselevich 2024-10-16 14:55:49 +02:00 • committed by clbot
parent da9a6e5b78
commit 7e78ebe796
1 changed files with 1 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

5
tvix/glue/src/builtins/import.rs
View file

@ -162,10 +162,7 @@ mod import_builtins {
.select("sha256") .select("sha256")
.map(|h| { .map(|h| {
h.to_str().and_then(|expected| { h.to_str().and_then(|expected| {
match nix_compat::nixhash::from_str( match nix_compat::nixhash::from_str(expected.to_str()?, Some("sha256")) {
expected.into_bstring().to_str()?,
Some("sha256"),
) {
Ok(NixHash::Sha256(digest)) => Ok(digest), Ok(NixHash::Sha256(digest)) => Ok(digest),
Ok(_) => unreachable!(), Ok(_) => unreachable!(),
Err(_e) => { Err(_e) => {