On Sun, 2026-03-08 at 21:10 +0200, Vladimir Oltean wrote:
> On Sun, Mar 08, 2026 at 11:40:44AM -0700, Joe Perches wrote:
> > Perhaps if matching only the include uses:
> > (/ is escaped because get_maintainer is stupid)
> > something like:
> >
> > K: include\s*\<linux\/phy\/phy(?:-common-props|-provider)?\.h\>
> Why is get_maintainer stupid?
The get_maintainer code used to match keywords is
foreach my $line (keys %keyword_hash) {
if ($text =~ m/$keyword_hash{$line}/x) {
so it seems the first / in the K: <foo> regex would terminate
the match.
It might have been better to use a different delimiter.
Maybe:
if ($text =~ m{$keyword_hash{$line}}/x
