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

Reply via email to