> -----Original Message----- > From: Intel-wired-lan <[email protected]> On Behalf > Of Mauro Carvalho Chehab > Sent: Thursday, January 29, 2026 9:08 AM > To: Jonathan Corbet <[email protected]>; Linux Doc Mailing List <linux- > [email protected]> > Cc: Mauro Carvalho Chehab <[email protected]>; > [email protected]; [email protected]; linux- > [email protected]; [email protected]; > [email protected]; Mauro Carvalho Chehab <[email protected]>; > Randy Dunlap <[email protected]> > Subject: [Intel-wired-lan] [PATCH v3 03/30] docs: kdoc_parser: move > var transformers to the beginning > > Just like functions and structs had their transform variables placed > at the beginning, move variable transforms to there as well. > > No functional changes. > > Signed-off-by: Mauro Carvalho Chehab <[email protected]> > Acked-by: Randy Dunlap <[email protected]> > Tested-by: Randy Dunlap <[email protected]> > --- > tools/lib/python/kdoc/kdoc_parser.py | 23 +++++++++++++---------- > 1 file changed, 13 insertions(+), 10 deletions(-) > > diff --git a/tools/lib/python/kdoc/kdoc_parser.py > b/tools/lib/python/kdoc/kdoc_parser.py > index fd57944ae907..0b68b140cd02 100644 > --- a/tools/lib/python/kdoc/kdoc_parser.py > +++ b/tools/lib/python/kdoc/kdoc_parser.py > @@ -191,6 +191,18 @@ function_xforms = [ > > (KernRe(r"__attribute__\s*\(\((?:[\w\s]+(?:\([^)]*\))?\s*,?)+\)\)\s+") > , ""), ] > > +# > +# Transforms for variable prototypes > +# > +var_xforms = [ > + (KernRe(r"__read_mostly"), ""), > + (KernRe(r"__ro_after_init"), ""), > + (KernRe(r"(?://.*)$"), ""), > + (KernRe(r"(?:/\*.*\*/)"), ""), > + (KernRe(r";$"), ""), > + (KernRe(r"=.*"), ""), > +] > + > # > # Ancillary functions > # > @@ -971,15 +983,6 @@ class KernelDoc: > ] > OPTIONAL_VAR_ATTR = "^(?:" + "|".join(VAR_ATTRIBS) + ")?" > > - sub_prefixes = [ > - (KernRe(r"__read_mostly"), ""), > - (KernRe(r"__ro_after_init"), ""), > - (KernRe(r"(?://.*)$"), ""), > - (KernRe(r"(?:/\*.*\*/)"), ""), > - (KernRe(r";$"), ""), > - (KernRe(r"=.*"), ""), > - ] > - > # > # Store the full prototype before modifying it > # > @@ -1003,7 +1006,7 @@ class KernelDoc: > # Drop comments and macros to have a pure C prototype > # > if not declaration_name: > - for r, sub in sub_prefixes: > + for r, sub in var_xforms: > proto = r.sub(sub, proto) > > proto = proto.rstrip() > -- > 2.52.0
Reviewed-by: Aleksandr Loktionov <[email protected]>
