diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2024-02-14 14:29:38 +0100 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2024-02-21 13:44:21 -0700 |
| commit | 8e93cb78a4f68c3c3c718e5388f0593227101393 (patch) | |
| tree | a6314bb6f67f2cd0a454718e302d513edc30bd26 | |
| parent | 27103dddc2dafd32de3ac04f4ab94478f083f1ac (diff) | |
| download | lwn-8e93cb78a4f68c3c3c718e5388f0593227101393.tar.gz lwn-8e93cb78a4f68c3c3c718e5388f0593227101393.zip | |
kernel-doc: handle #if in enums as well
In addition to #ifdef, #define and #endif, also handle
any #if since we may be using e.g. #if IS_ENABLED(...).
I didn't find any instances of this in the kernel now,
there are enums with such ifs inside, but I didn't find
any with kernel-doc as well. However, it came up as we
were adding such a construct in our driver and warnings
from kernel-doc were the result.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Link: https://lore.kernel.org/r/20240214142937.80ee86a3beae.Ibcc5bd97a20cd10a792663e4b254cd46c7e8b520@changeid
| -rwxr-xr-x | scripts/kernel-doc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/kernel-doc b/scripts/kernel-doc index 7acb7554abb9..71a89d8832c1 100755 --- a/scripts/kernel-doc +++ b/scripts/kernel-doc @@ -1328,7 +1328,7 @@ sub dump_enum($$) { $x =~ s@/\*.*?\*/@@gos; # strip comments. # strip #define macros inside enums - $x =~ s@#\s*((define|ifdef)\s+|endif)[^;]*;@@gos; + $x =~ s@#\s*((define|ifdef|if)\s+|endif)[^;]*;@@gos; if ($x =~ /typedef\s+enum\s*\{(.*)\}\s*(\w*)\s*;/) { $declaration_name = $2; |