On Wed, 2023-09-06 at 14:45 +0800, David Gow wrote:
> On Mon, 4 Sept 2023 at 21:22, <benj...@sipsolutions.net> wrote:
> >
> > From: Benjamin Berg <benjam...@intel.com>
> >
> > The existing KUNIT_ARRAY_PARAM macro requires a separate function
> > to
> > get the description. However, in a lot of cases the description can
> > just be copied directly from the array. Add a second macro that
> > avoids having to write a static function just for a single strscpy.
> >
> > Signed-off-by: Benjamin Berg <benjam...@intel.com>
> > ---
>
> Looks good to me: this will be much more convenient. The actual
> implementation looks spot on, just a small comment about the
> documentation change.
>
> It may make sense to write some tests and/or some follow-up patches to
> existing tests to use this macro, too. I'm just a little wary of
> introducing something totally unused. (I'm happy to do these myself if
> you don't have time, though.)
I agree. I am happy to submit one or more patches to change the
existing users. The question would be how we pull such a change in.
Should it be submitted separately for each subtree or can we pull them
all in at the same time here?
Benjamin
Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928