mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(users/Profpatsch/netencode): use write_all in encoder

Browse source 
`write` returns the written usize; now I wonder why rustc didn’t at
least produce a warning because the result was unused. Do we need to
add any flags to `rustSimple`?

Change-Id: If8d51d95c993dec6c92e46dbc82cd8cdd398f441
Reviewed-on: https://cl.tvl.fyi/c/depot/+/3056
Tested-by: BuildkiteCI
Reviewed-by: sterni <sternenseemann@systemli.org>
This commit is contained in:
Profpatsch 2021-04-24 00:59:48 +02:00
parent 17239c597d
commit 3ef9b8fbd7
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
users/Profpatsch/netencode/netencode.rs
View file

@ -158,12 +158,12 @@ pub fn encode<W: Write>(w: &mut W, u: &U) -> std::io::Result<()> {
U::I7(i) => write!(w, "i7:{},", i),
U::Text(s) => {
write!(w, "t{}:", s.len());
w.write(s.as_bytes());
w.write_all(s.as_bytes());
write!(w, ",")
}
U::Binary(s) => {
write!(w, "b{}:", s.len());
w.write(&s);
w.write_all(&s);
write!(w, ",")
},
U::Sum(Tag{tag, val}) => encode_tag(w, tag, val),
@ -173,7 +173,7 @@ pub fn encode<W: Write>(w: &mut W, u: &U) -> std::io::Result<()> {
encode_tag(&mut c, k, v)?;
}
write!(w, "{{{}:", c.get_ref().len())?;
w.write(c.get_ref())?;
w.write_all(c.get_ref())?;
write!(w, "}}")
},
U::List(l) => {
@ -182,7 +182,7 @@ pub fn encode<W: Write>(w: &mut W, u: &U) -> std::io::Result<()> {
encode(&mut c, u)?;
}
write!(w, "[{}:", c.get_ref().len())?;
w.write(c.get_ref())?;
w.write_all(c.get_ref())?;
write!(w, "]")
}
}