diff options
author | Johannes Berg <johannes.berg@intel.com> | 2013-11-12 18:11:12 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-11-12 22:09:32 -0500 |
commit | e946c43a116526e3d947dc03aeb165c1effd9f8f (patch) | |
tree | 9118f9c6ec4a286e3be3d3e6c38fee8752b6966d /scripts/kernel-doc | |
parent | c7708649ccbdc21e85c95ca1a01b28342d939d39 (diff) |
kernel-doc: improve "no structured comments found" error
When using '!Ffile function' in a docbook template, and the function no
longer exists, you get a "no structured comments found" error from the
kernel-doc processing script. It's useful to know which functions it was
looking for, so print them out in this case. Also do the same for '!Pfile
doc-section'
The same error also happens when using '!Efile' when some exported
functions aren't documented (in the same file.) There's a very large
number of such functions though, so don't print the message in this case
-- right now it would give ~850 messages.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Cc: Rob Landley <rob@landley.net>
Cc: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/kernel-doc')
-rwxr-xr-x | scripts/kernel-doc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/scripts/kernel-doc b/scripts/kernel-doc index 4305b2f2ec5e..dbd3e1ebbdad 100755 --- a/scripts/kernel-doc +++ b/scripts/kernel-doc | |||
@@ -257,6 +257,7 @@ my $man_date = ('January', 'February', 'March', 'April', 'May', 'June', | |||
257 | 'July', 'August', 'September', 'October', | 257 | 'July', 'August', 'September', 'October', |
258 | 'November', 'December')[(localtime)[4]] . | 258 | 'November', 'December')[(localtime)[4]] . |
259 | " " . ((localtime)[5]+1900); | 259 | " " . ((localtime)[5]+1900); |
260 | my $show_not_found = 0; | ||
260 | 261 | ||
261 | # Essentially these are globals. | 262 | # Essentially these are globals. |
262 | # They probably want to be tidied up, made more localised or something. | 263 | # They probably want to be tidied up, made more localised or something. |
@@ -369,6 +370,8 @@ while ($ARGV[0] =~ m/^-(.*)/) { | |||
369 | usage(); | 370 | usage(); |
370 | } elsif ($cmd eq '-no-doc-sections') { | 371 | } elsif ($cmd eq '-no-doc-sections') { |
371 | $no_doc_sections = 1; | 372 | $no_doc_sections = 1; |
373 | } elsif ($cmd eq '-show-not-found') { | ||
374 | $show_not_found = 1; | ||
372 | } | 375 | } |
373 | } | 376 | } |
374 | 377 | ||
@@ -2536,6 +2539,9 @@ sub process_file($) { | |||
2536 | } | 2539 | } |
2537 | if ($initial_section_counter == $section_counter) { | 2540 | if ($initial_section_counter == $section_counter) { |
2538 | print STDERR "Warning(${file}): no structured comments found\n"; | 2541 | print STDERR "Warning(${file}): no structured comments found\n"; |
2542 | if (($function_only == 1) && ($show_not_found == 1)) { | ||
2543 | print STDERR " Was looking for '$_'.\n" for keys %function_table; | ||
2544 | } | ||
2539 | if ($output_mode eq "xml") { | 2545 | if ($output_mode eq "xml") { |
2540 | # The template wants at least one RefEntry here; make one. | 2546 | # The template wants at least one RefEntry here; make one. |
2541 | print "<refentry>\n"; | 2547 | print "<refentry>\n"; |