Re: [PATCH v2 2/3] list_debug: Introduce inline wrappers for debug checks

From: Steven Rostedt
Date: Fri Aug 04 2023 - 13:59:13 EST


On Fri, 4 Aug 2023 13:57:57 -0400
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> On Fri, 4 Aug 2023 19:49:48 +0200
> Marco Elver <elver@xxxxxxxxxx> wrote:
>
> > > I've been guilty of this madness myself, but I have learned the errors of
> > > my ways, and have been avoiding doing so in any new code I write.
> >
> > That's fair. We can call them __list_*_valid() (inline), and
> > __list_*_valid_or_report() ?
>
> __list_*_valid_check() ?
>

I have to admit, I think the main reason kernel developers default to using
these useless underscores is because kernel developers are notoriously
lousy at naming. ;-)

-- Steve