diff options
author | Chen-Yu Tsai <wenst@chromium.org> | 2024-10-08 16:29:04 +0800 |
---|---|---|
committer | Jonathan Corbet <corbet@lwn.net> | 2024-10-14 10:19:22 -0600 |
commit | be9264110e4e874622d588a75daf930539fdf6ea (patch) | |
tree | 6c9805ee59790cbe43ec4058e4263c3fdfc6e04d | |
parent | 6a32c8dfec8126040a3f5484f4f7301ca96d697c (diff) | |
download | lwn-be9264110e4e874622d588a75daf930539fdf6ea.tar.gz lwn-be9264110e4e874622d588a75daf930539fdf6ea.zip |
scripts/kernel-doc: Do not track section counter across processed files
The section counter tracks how many sections of kernel-doc were added.
The only real use of the counter value is to check if anything was
actually supposed to be output and give a warning is nothing is
available.
The current logic of remembering the initial value and then resetting
the value then when processing each file means that if a file has the
same number of sections as the previously processed one, a warning is
incorrectly given.
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Link: https://lore.kernel.org/r/20241008082905.4005524-1-wenst@chromium.org
-rwxr-xr-x | scripts/kernel-doc | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/scripts/kernel-doc b/scripts/kernel-doc index 8728f9824c65..a84510d3cd1f 100755 --- a/scripts/kernel-doc +++ b/scripts/kernel-doc @@ -2322,7 +2322,6 @@ sub process_inline($$) { sub process_file($) { my $file; - my $initial_section_counter = $section_counter; my ($orig_file) = @_; $file = map_filename($orig_file); @@ -2360,8 +2359,7 @@ sub process_file($) { } # Make sure we got something interesting. - if ($initial_section_counter == $section_counter && $ - output_mode ne "none") { + if (!$section_counter && $output_mode ne "none") { if ($output_selection == OUTPUT_INCLUDE) { emit_warning("${file}:1", "'$_' not found\n") for keys %function_table; |