On Mon, 2017-11-27 at 23:47 +0100, Martin Wilck wrote: > + /* This zero-pads the destination */ > + strncpy(to, from, to_length); > + if (from_length < to_length && !compatible) > + /* > + * space pad the string if it is short. > + */ > + memset(&to[from_length], ' ', to_length - from_length);
Since the code block controlled by the if-statement consists of multiple
lines, shouldn't that block be surrounded by braces ({})? Anyway:
Reviewed-by: Bart Van Assche <bart.vanass...@wdc.com>
