Skip to content

Commit 00a9173

Browse files
Merge pull request #21648 from alexanderkjall/borsh-error-compilation-failure
fix smol_str compilation error
2 parents 4d29f32 + f9ca8b2 commit 00a9173

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

lib/smol_str/src/borsh.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,9 @@ impl BorshDeserialize for SmolStr {
2929
}))
3030
} else {
3131
// u8::vec_from_reader always returns Some on success in current implementation
32-
let vec = u8::vec_from_reader(len, reader)?
33-
.ok_or_else(|| Error::other("u8::vec_from_reader unexpectedly returned None"))?;
32+
let vec = u8::vec_from_reader(len, reader)?.ok_or_else(|| {
33+
Error::new(ErrorKind::Other, "u8::vec_from_reader unexpectedly returned None")
34+
})?;
3435
Ok(SmolStr::from(String::from_utf8(vec).map_err(|err| {
3536
let msg = err.to_string();
3637
Error::new(ErrorKind::InvalidData, msg)

lib/smol_str/tests/test.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ mod test_str_ext {
393393
}
394394
}
395395

396-
#[cfg(feature = "borsh")]
396+
#[cfg(all(feature = "borsh", feature = "std"))]
397397
mod borsh_tests {
398398
use borsh::BorshDeserialize;
399399
use smol_str::{SmolStr, ToSmolStr};

0 commit comments

Comments
 (0)