Skip to content

Commit a8e2add

Browse files
authored
Merge pull request #21764 from Veykril/push-wxmovumvpzqm
smol_str: Improve API surface
2 parents 23cc64a + 8ff72bb commit a8e2add

4 files changed

Lines changed: 247 additions & 44 deletions

File tree

lib/smol_str/src/borsh.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ impl BorshDeserialize for SmolStr {
1616
#[inline]
1717
fn deserialize_reader<R: Read>(reader: &mut R) -> borsh::io::Result<Self> {
1818
let len = u32::deserialize_reader(reader)?;
19-
if (len as usize) < INLINE_CAP {
19+
if (len as usize) <= INLINE_CAP {
2020
let mut buf = [0u8; INLINE_CAP];
2121
reader.read_exact(&mut buf[..len as usize])?;
2222
_ = core::str::from_utf8(&buf[..len as usize]).map_err(|err| {
@@ -29,9 +29,8 @@ 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)?.ok_or_else(|| {
33-
Error::new(ErrorKind::Other, "u8::vec_from_reader unexpectedly returned None")
34-
})?;
32+
let vec = u8::vec_from_reader(len, reader)?
33+
.ok_or_else(|| Error::other("u8::vec_from_reader unexpectedly returned None"))?;
3534
Ok(SmolStr::from(String::from_utf8(vec).map_err(|err| {
3635
let msg = err.to_string();
3736
Error::new(ErrorKind::InvalidData, msg)

0 commit comments

Comments
 (0)