On Sun, Jan 3, 2016 at 6:27 PM, Arne Schwabe <[email protected]> wrote:
> + /* did compression save us anything? Include 2 byte compression header
> + in calculation */
> + {
> + if (compressed && work.len + 2 < buf->len)
> + {
> + ASSERT(buf_prepend(&work, 2));
> + uint8_t *head = BPTR (&work);
> + head[0] = COMP_ALGV2_INDICATOR_BYTE;
> + head[1] = COMP_ALGV2_LZ4_BYTE;
> + *buf = work;
> + }
> + else
> + compv2_escape_data_ifneeded(buf);
> + }
I almost don't dare to say it - but I meant the outer braces
surrounding this entire if () {} else {}-block, not the braces of the
else-block (which I consider good practice). But, as just discussed
on IRC, Gert could just take that when applying :)
So, ACK for everything except the braces ;)
-Steffan
