Jeff King <peff@xxxxxxxx> writes: >> +/* >> + * Convert a four hex digit packet line length header into its numeric >> + * representation. linelen should not be null-terminated. > > Minor nit, but it is perfectly fine if there is a NUL. Maybe "linelen > does not need to be..."? Yup, I was wondering about the same thing. I actually would go stronger than "does not need to be", as the byte after these four would never be taken as "the terminator for the linelen bytes", even it happens to be '\0'. Just getting rid of the extra sentence would suffice for that, I think. The first sentence makes it clear that it is about interpreting the 4 bytes we are given, and those 4 bytes come from the 'packet line length header' the caller has.
