Hello Bruce, On Mon, May 13, 2024 at 1:37 PM Bruce Richardson <bruce.richard...@intel.com> wrote: > > @@ -16,6 +16,8 @@ > > #include "eal_firmware.h" > > #include "eal_private.h" > > > > +static const char * const compression_algorithms[] = { "xz", "zst" }; > Very minor nit: these are the suffixes used to find the files, more than > they are the compression algorithms. There I'd suggest renaming to > "compression_suffixes". > [Perhaps just fix on apply if taking this suggestion]
I'll fix when applying. Thanks for the review and tests. -- David Marchand