forked from luck/tmp_suning_uos_patched
kernel-doc: concatenate contents of colliding sections
If there are multiple sections with the same section name, the current implementation results in several sections by the same heading, with the content duplicated from the last section to all. Even if there's the error message, a more graceful approach is to combine all the identically named sections into one, with concatenated contents. With the supported sections already limited to select few, there are massively fewer collisions than there used to be, but this is still useful for e.g. when function parameters are documented in the middle of a documentation comment, with description spread out above and below. (This is not a recommended documentation style, but used in the kernel nonetheless.) We can now also demote the error to a warning. Signed-off-by: Jani Nikula <jani.nikula@intel.com>
This commit is contained in:
parent
f624adef3d
commit
32217761ee
|
@ -524,11 +524,13 @@ sub dump_section {
|
|||
} else {
|
||||
# print STDERR "other section '$name' = '$contents'\n";
|
||||
if (defined($sections{$name}) && ($sections{$name} ne "")) {
|
||||
print STDERR "${file}:$.: error: duplicate section name '$name'\n";
|
||||
++$errors;
|
||||
print STDERR "${file}:$.: warning: duplicate section name '$name'\n";
|
||||
++$warnings;
|
||||
$sections{$name} .= $contents;
|
||||
} else {
|
||||
$sections{$name} = $contents;
|
||||
push @sectionlist, $name;
|
||||
}
|
||||
$sections{$name} = $contents;
|
||||
push @sectionlist, $name;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user