diff --git a/components/net/sf/briar/serial/WriterImpl.java b/components/net/sf/briar/serial/WriterImpl.java index 42083227a190099cf425246ea9f84002d04b40c9..1a9359aec186bd5f8f6e761dcc96b5c2956bee96 100644 --- a/components/net/sf/briar/serial/WriterImpl.java +++ b/components/net/sf/briar/serial/WriterImpl.java @@ -125,12 +125,10 @@ class WriterImpl implements Writer { } private void writeLength(int i) throws IOException { - if(i >= 0 && i <= Byte.MAX_VALUE) - writeUint7((byte) i); - else if(i >= Byte.MIN_VALUE && i <= Byte.MAX_VALUE) - writeInt8((byte) i); - else if(i >= Short.MIN_VALUE && i <= Short.MAX_VALUE) - writeInt16((short) i); + assert i >= 0; + // Fun fact: it's never worth writing a length as an int8 + if(i <= Byte.MAX_VALUE) writeUint7((byte) i); + else if(i <= Short.MAX_VALUE) writeInt16((short) i); else writeInt32(i); }