[dpdk-dev] [PATCH v3] log: register with standardized names
Thomas Monjalon
thomas at monjalon.net
Tue May 11 14:10:47 CEST 2021
11/05/2021 13:59, Bruce Richardson:
> On Tue, May 11, 2021 at 12:09:27PM +0200, Thomas Monjalon wrote:
> > --- a/drivers/meson.build
> > +++ b/drivers/meson.build
> > @@ -59,6 +59,11 @@ foreach subpath:subdirs
> > drivers += driver_path[1]
> > else
> > class = subpath
> > + endif
> > +
> > + log_prefix = 'pmd.' + class
> > +
> > + if drivers.length() == 0
> > subdir(class)
> > endif
> >
>
> Minor nit - rather than splitting the existing block here, I suggest
> setting log_prefix = '' at the start of the loop, and leaving the existing
> else as-is. Then after the "subdir(class)" just check if log_prefix ==
> '' and then set it to the default. I think it might be easier to read that
> way. [See suggested change below (untested)]
>
> /Bruce
>
> diff --git a/drivers/meson.build b/drivers/meson.build
> index 0052247a5..9d518d361 100644
> --- a/drivers/meson.build
> +++ b/drivers/meson.build
> @@ -51,6 +51,7 @@ endif
> foreach subpath:subdirs
> drivers = []
> std_deps = []
> + log_prefix = ''
>
> # subpath can be either "class" or "class/driver"
> if subpath.contains('/')
> @@ -62,6 +63,10 @@ foreach subpath:subdirs
> subdir(class)
> endif
>
> + if log_prefix == '':
> + log_prefix = 'pmd.' + class
> + endif
> +
OK with this change, I'll send a v4.
Nit: please remove useless huge context when replying :)
More information about the dev
mailing list