From: Chen-Yu Tsai <wenst@chromium.org>
Date: Tue, 8 Oct 2024 08:29:04 +0000 (+0800)
Subject: scripts/kernel-doc: Do not track section counter across processed files
X-Git-Tag: microblaze-v6.16~536^2~42
X-Git-Url: http://git.monstr.eu/?a=commitdiff_plain;h=be9264110e4e874622d588a75daf930539fdf6ea;p=linux-2.6-microblaze.git

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
---

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;