From: Jakub Kicinski
> Sent: 27 July 2017 10:26
...
> > - use snprintf().
>
> To effectively print an integer into an amply sized array? I need to
> guarantee that the string will fit otherwise I would request a FW image
> with a wrong name. snprintf() would only mask such a bug.
Eh?
If, for any reason, the buffer isn't long enough snprintf() won't
write over random memory - sprint() will, and you may not notice.
> > - kill those hh, the arguments are of type 'int'.
>
> It doesn't matter. I will be more careful in the future, though.
>
> > In fact make 'interface' 'unsigned int' as well.
>
> It's a value read from the hardware, and it's 16 bits wide, therefore
> my preference it to explicitly size the variable.
Right, so you keep asking the compiler to generate code to mask
any arithmetic results (written into the variable) back down to
16 bits.
OTOH, looking at some of the functions in nfp_cppcore.c you don't
care about performance at all.
David
