Hi Brandon, On Tue, 27 Dec 2022 at 11:45, Maier, Brandon L Collins <brandon.ma...@collins.com> wrote: > > Hi Sean, > > > From: Sean Anderson <sean.ander...@seco.com> > > On 12/27/22 09:46, Brandon Maier wrote: > > > +static const unsigned long zstd_compressed_size = 197; > > > > You can use ARRAY_SIZE (or sizeof) for this. > > sizeof() - 1 as it's a string, but I had put the size here manually as that's > what all the other tests do. I can switch them all to sizeof() if that is > preferred.
Yes I think it is better too, but please change the others in a separate patch. Regards, Simon