refactor(tvix/value): replace static representation with SmolStr r/4498

The only uses of the static variant were for `"name"` and `"value"`,
which are both small enough to fit into a SmolStr. The size of
NixString accomodates `String` anyways, so we may as well inline them.

Additionally smol_str is already in the dependency graph because rnix
uses it, and using it for representations of identifiers is sensible.

Change-Id: I9969312256d1657d69128e54c47dc7294a18ce58
Reviewed-on: https://cl.tvl.fyi/c/depot/+/6165
Tested-by: BuildkiteCI
Reviewed-by: sterni <sternenseemann@systemli.org>
Reviewed-by: grfn <grfn@gws.fyi>

1 files changed, 1 insertions, 0 deletions

diff --git a/tvix/eval/Cargo.toml b/tvix/eval/Cargo.toml
index 4c1cf4f30223..48156dc1e7b0 100644
--- a/tvix/eval/Cargo.toml
+++ b/tvix/eval/Cargo.toml
@@ -7,6 +7,7 @@ edition = "2021"
 
 [dependencies]
 rnix = "0.10.2"
+smol_str = "0.1"
 
 [dev-dependencies]
 criterion = "0.3.6"