On 15/02/2025 04:11, Jakub Kicinski wrote: > On Thu, 13 Feb 2025 12:00:25 +0100 Peter Seiderer wrote: >> Use defines for the various dec/hex number parsing digits lengths >> (hex32_arg/num_arg calls). > > I don't understand the value of this patch, TBH. > > Example: > > +#define HEX_2_DIGITS 2 > > - len = hex32_arg(&user_buffer[i], 2, &tmp_value); > + len = hex32_arg(&user_buffer[i], HEX_2_DIGITS, &tmp_value); > > The word hex is already there. > There is still a two. > I don't think the new define has any explanatory power? Seems like the intent of the code is that the argument is one byte, which just happens to take two digits to represent in hex. Perhaps that would help to come up with more meaningful names for these constants? (Can't think of good ones off the top of my head)
