[PATCH v2 2/2] test: compression: add zstd uncompression test

Simon Glass sjg at chromium.org
Thu Dec 29 23:38:44 CET 2022


Hi Brandon,

On Tue, 27 Dec 2022 at 11:45, Maier, Brandon L Collins
<Brandon.Maier at collins.com> wrote:
>
> Hi Sean,
>
> > From: Sean Anderson <sean.anderson at 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


More information about the U-Boot mailing list