On Wed, Nov 3, 2021 at 11:27 AM David Hunt <david.h...@intel.com> wrote:
>
> >> for (; i >= 0; i--)
> >> - if (coremask[i] != '0')
> >> + if (coremask[i] != '0') {
> >> + RTE_LOG(ERR, EAL, "invalid characters in coremask:
> >> [%s]\n",
> >> + coremask_orig);
> > Nit: capital letter.
>
>
> Many other instances of 'invalid' in this file, will I change them all
> to "Invalid", or just this one?Let's be consistent for additions of this patch. Thanks. -- David Marchand
