Jakub Narebski <jnareb@xxxxxxxxx> writes:
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index 8c1e02c..5d424e5 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -3140,8 +3140,8 @@ sub git_patchset_body {
> # compact combined diff output can have some patches skipped
> # find which patch (using pathname of result) we are at now
> my $to_name;
> - if ($patch_line =~ m!^diff --cc "?(.*)"?$!) {
> - $to_name = $1;
> + if ($patch_line =~ m!^diff --cc ("?.*"?)$!) {
> + $to_name = unquote($1);
> }
This patch obviously makes sense, but I wonder why you use the
form m!foobar! here, instead of far simpler to read /foobar/
form?
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
