On Sun, Apr 23, 2017 at 2:34 PM, brian m. carlson <sandals@xxxxxxxxxxxxxxxxxxxx> wrote: > Convert as many instances of unsigned char [20] as possible, Update the s/Update/update/ or s/,/./ Side remark: In all patches up to now you put a space between the char and the [20] (or [40]), which is irritating to read (for me). I presume putting it adjacent to the char would offend others as that would violate our coding style. So having the whitespace in between "char" and the array makes sense. Up and including this patch looks good. Thanks, Stefan