get_maintainer: add subsystem to reviewer output
Reviewer output currently does not include the subsystem that matched. Add it. Miscellanea: o Add a get_subsystem_name routine to centralize this Signed-off-by: Joe Perches <joe@perches.com> Tested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> Cc: Lee Jones <lee.jones@linaro.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
4f07510df2
commit
2a7cb1dc82
|
@ -974,6 +974,20 @@ sub find_ending_index {
|
|||
return $index;
|
||||
}
|
||||
|
||||
sub get_subsystem_name {
|
||||
my ($index) = @_;
|
||||
|
||||
my $start = find_starting_index($index);
|
||||
|
||||
my $subsystem = $typevalue[$start];
|
||||
if ($output_section_maxlen && length($subsystem) > $output_section_maxlen) {
|
||||
$subsystem = substr($subsystem, 0, $output_section_maxlen - 3);
|
||||
$subsystem =~ s/\s*$//;
|
||||
$subsystem = $subsystem . "...";
|
||||
}
|
||||
return $subsystem;
|
||||
}
|
||||
|
||||
sub get_maintainer_role {
|
||||
my ($index) = @_;
|
||||
|
||||
|
@ -982,12 +996,7 @@ sub get_maintainer_role {
|
|||
my $end = find_ending_index($index);
|
||||
|
||||
my $role = "unknown";
|
||||
my $subsystem = $typevalue[$start];
|
||||
if ($output_section_maxlen && length($subsystem) > $output_section_maxlen) {
|
||||
$subsystem = substr($subsystem, 0, $output_section_maxlen - 3);
|
||||
$subsystem =~ s/\s*$//;
|
||||
$subsystem = $subsystem . "...";
|
||||
}
|
||||
my $subsystem = get_subsystem_name($index);
|
||||
|
||||
for ($i = $start + 1; $i < $end; $i++) {
|
||||
my $tv = $typevalue[$i];
|
||||
|
@ -1021,16 +1030,7 @@ sub get_maintainer_role {
|
|||
sub get_list_role {
|
||||
my ($index) = @_;
|
||||
|
||||
my $i;
|
||||
my $start = find_starting_index($index);
|
||||
my $end = find_ending_index($index);
|
||||
|
||||
my $subsystem = $typevalue[$start];
|
||||
if ($output_section_maxlen && length($subsystem) > $output_section_maxlen) {
|
||||
$subsystem = substr($subsystem, 0, $output_section_maxlen - 3);
|
||||
$subsystem =~ s/\s*$//;
|
||||
$subsystem = $subsystem . "...";
|
||||
}
|
||||
my $subsystem = get_subsystem_name($index);
|
||||
|
||||
if ($subsystem eq "THE REST") {
|
||||
$subsystem = "";
|
||||
|
@ -1118,7 +1118,8 @@ sub add_categories {
|
|||
}
|
||||
}
|
||||
if ($email_reviewer) {
|
||||
push_email_addresses($pvalue, 'reviewer');
|
||||
my $subsystem = get_subsystem_name($i);
|
||||
push_email_addresses($pvalue, "reviewer:$subsystem");
|
||||
}
|
||||
} elsif ($ptype eq "T") {
|
||||
push(@scm, $pvalue);
|
||||
|
|
Loading…
Reference in New Issue