[PATCH 1/3] staging: vt6656: Use ARRAY_SIZE instead of define RATE_54M
Oscar Carter
oscar.carter at gmx.com
Mon Apr 6 16:27:42 UTC 2020
On Mon, Apr 06, 2020 at 02:13:23PM +0300, Dan Carpenter wrote:
> On Sat, Apr 04, 2020 at 04:13:58PM +0200, Oscar Carter wrote:
> > Use ARRAY_SIZE to replace the define RATE_54M so we will never have a
> > mismatch. In this way, avoid the possibility of a buffer overflow if
> > this define is changed in the future to a greater value.
> >
>
> Future proofing is not really a valid reason to change this.
Ok, then I leave it as is.
> We have to assume that future programmers are not idiots.
>
That was not my intention. I'm sorry.
> The only valid reason to do this is readability, but I'm not convinced
> the new version is more readable.
>
Ok.
> regards,
> dan carpenter
>
Thanks,
oscar carter
More information about the devel
mailing list