about summary refs log tree commit diff
path: root/third_party/sources
diff options
context:
space:
mode:
authorFlorian Klink <flokli@flokli.de>2024-04-22T20·16+0300
committerclbot <clbot@tvl.fyi>2024-04-22T20·28+0000
commitff9e8743f6572bc44c084e49adc13bf293016295 (patch)
tree09c8d34db2c80ab8a4df57f7dd031307397b1404 /third_party/sources
parent6d79cf39b9b9a2d250e1a94ac6f36921cea298ee (diff)
fix(tvix/eval): don't impl From<NixString> for String r/7991
This caught me by accident in an earlier revision of cl/11500 -
I had a `NixString`, wanted to return it as a `String`, so I was naively
calling `s.into()`.

That unfortunately gave me the `Display` implementation of `NixString`,
which quotes strings, causing an annoying error further up the stack.

NixStrings are bytes, we can keep the impl From<NixString> for BString,
but having a `.into()` suddenly do quoting is more than unexpected.

Change-Id: I5434ba94bfe6c493d0a57e68225ecc22daa4b948
Reviewed-on: https://cl.tvl.fyi/c/depot/+/11505
Autosubmit: flokli <flokli@flokli.de>
Reviewed-by: tazjin <tazjin@tvl.su>
Reviewed-by: raitobezarius <tvl@lahfa.xyz>
Tested-by: BuildkiteCI
Diffstat (limited to 'third_party/sources')
0 files changed, 0 insertions, 0 deletions