diff options
author | James Landrein <james.landrein@gmail.com> | 2022-10-25T11·56+0200 |
---|---|---|
committer | clbot <clbot@tvl.fyi> | 2022-10-28T10·21+0000 |
commit | ccab9c06a2f01353b8a91469d46a498a75961431 (patch) | |
tree | c92cc00b809b24cd5da2f52aa4cb70ae728ad84a /tvix | |
parent | 3aec6786798d6ffd5e5eb5b205827b92b6185f1d (diff) |
feat(tvix/eval): add builtins.replaceStrings r/5219
Change-Id: I93dcdaeb101364ee2273bcaeb19acb57cf6b9e7d Reviewed-on: https://cl.tvl.fyi/c/depot/+/7034 Autosubmit: j4m3s <james.landrein@gmail.com> Reviewed-by: Adam Joseph <adam@westernsemico.com> Reviewed-by: tazjin <tazjin@tvl.su> Tested-by: BuildkiteCI
Diffstat (limited to 'tvix')
-rw-r--r-- | tvix/eval/src/builtins/mod.rs | 71 | ||||
-rw-r--r-- | tvix/eval/src/tests/nix_tests/eval-okay-replacestrings.exp (renamed from tvix/eval/src/tests/nix_tests/notyetpassing/eval-okay-replacestrings.exp) | 0 | ||||
-rw-r--r-- | tvix/eval/src/tests/nix_tests/eval-okay-replacestrings.nix (renamed from tvix/eval/src/tests/nix_tests/notyetpassing/eval-okay-replacestrings.nix) | 0 | ||||
-rw-r--r-- | tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.exp | 1 | ||||
-rw-r--r-- | tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.nix | 5 |
5 files changed, 77 insertions, 0 deletions
diff --git a/tvix/eval/src/builtins/mod.rs b/tvix/eval/src/builtins/mod.rs index 7dd7ee946d32..32626daa9b27 100644 --- a/tvix/eval/src/builtins/mod.rs +++ b/tvix/eval/src/builtins/mod.rs @@ -554,6 +554,77 @@ fn pure_builtins() -> Vec<Builtin> { Ok(Value::attrs(NixAttrs::from_map(res))) }, ), + Builtin::new( + "replaceStrings", + &[true, true, true], + |args: Vec<Value>, vm: &mut VM| { + let from = args[0].to_list()?.into_iter(); + let to = args[1].to_list()?.into_iter(); + let string = args[2].to_str()?; + + let mut res = String::new(); + + let mut i: usize = 0; + let mut empty_string_replace = false; + + // This can't be implemented using Rust's string.replace() as + // well as a map because we need to handle errors with results + // as well as "reset" the iterator to zero for the replacement + // everytime there's a successful match. + // Also, Rust's string.replace allocates a new string + // on every call which is not preferable. + 'outer: while i < string.len() { + // Try a match in all the from strings + for elem in std::iter::zip(from.clone(), to.clone()) { + let from = elem.0.force(vm)?.to_str()?; + let to = elem.1.force(vm)?.to_str()?; + + if i + from.len() >= string.len() { + continue; + } + + // We already applied a from->to with an empty from + // transformation. + // Let's skip it so that we don't loop infinitely + if empty_string_replace && from.as_str().len() == 0 { + continue; + } + + // if we match the `from` string, let's replace + if &string[i..i + from.len()] == from.as_str() { + res += &to; + i += from.len(); + + // remember if we applied the empty from->to + empty_string_replace = from.as_str().len() == 0; + + continue 'outer; + } + } + + // If we don't match any `from`, we simply add a character + res += &string[i..i + 1]; + i += 1; + + // Since we didn't apply anything transformation, + // we reset the empty string replacement + empty_string_replace = false; + } + + // Special case when the string is empty or at the string's end + // and one of the from is also empty + for elem in std::iter::zip(from.clone(), to.clone()) { + let from = elem.0.force(vm)?.to_str()?; + let to = elem.1.force(vm)?.to_str()?; + + if from.as_str().len() == 0 { + res += &to; + break; + } + } + Ok(Value::String(res.into())) + }, + ), Builtin::new("seq", &[true, true], |mut args: Vec<Value>, _: &mut VM| { // The builtin calling infra has already forced both args for us, so we just return the // second and ignore the first diff --git a/tvix/eval/src/tests/nix_tests/notyetpassing/eval-okay-replacestrings.exp b/tvix/eval/src/tests/nix_tests/eval-okay-replacestrings.exp index 72e8274d8c58..72e8274d8c58 100644 --- a/tvix/eval/src/tests/nix_tests/notyetpassing/eval-okay-replacestrings.exp +++ b/tvix/eval/src/tests/nix_tests/eval-okay-replacestrings.exp diff --git a/tvix/eval/src/tests/nix_tests/notyetpassing/eval-okay-replacestrings.nix b/tvix/eval/src/tests/nix_tests/eval-okay-replacestrings.nix index bd8031fc004e..bd8031fc004e 100644 --- a/tvix/eval/src/tests/nix_tests/notyetpassing/eval-okay-replacestrings.nix +++ b/tvix/eval/src/tests/nix_tests/eval-okay-replacestrings.nix diff --git a/tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.exp b/tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.exp new file mode 100644 index 000000000000..c2cb89bac663 --- /dev/null +++ b/tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.exp @@ -0,0 +1 @@ +[ "fabir" "a" "1a1" ] diff --git a/tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.nix b/tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.nix new file mode 100644 index 000000000000..b8101c448bb0 --- /dev/null +++ b/tvix/eval/src/tests/tvix_tests/eval-okay-builtins-replaceStrings.nix @@ -0,0 +1,5 @@ +[ + (builtins.replaceStrings ["oo" "a"] ["a" "i"] "foobar") + (builtins.replaceStrings ["o"] ["a"] "a") + (builtins.replaceStrings ["" ""] ["1" "2"] "a") +] |