spelling: favor `publicly' over `publically'

[public-inbox.git] / lib / PublicInbox / SolverGit.pm

diff --git a/lib/PublicInbox/SolverGit.pm b/lib/PublicInbox/SolverGit.pm
index 8629f0dafeb4216b41ac2c0951c866c8e76c2dcf..06b4cdc40c220d6d849f9c05ef3b439cf0708471 100644 (file)
--- a/lib/PublicInbox/SolverGit.pm
+++ b/lib/PublicInbox/SolverGit.pm
@@ -5,7 +5,7 @@
 # searching inboxes for post-image blobs.
 
 # this emits a lot of debugging/tracing information which may be
-# publically viewed over HTTP(S).  Be careful not to expose
+# publicly viewed over HTTP(S).  Be careful not to expose
 # local filesystem layouts in the process.
 package PublicInbox::SolverGit;
 use strict;
@@ -169,7 +169,7 @@ sub extract_diff ($$) {
        my $patch = $9;
 
        # don't care for leading 'a/' and 'b/'
-       my (undef, @a) = split(m{/}, git_unquote($path_a));
+       my (undef, @a) = split(m{/}, git_unquote($path_a)) if defined($path_a);
        my (undef, @b) = split(m{/}, git_unquote($path_b));
 
        # get rid of path-traversal attempts and junk patches:
@@ -177,7 +177,7 @@ sub extract_diff ($$) {
        state $bad_component = { map { $_ => 1 } ('', '.', '..') };
        foreach (@a, @b) { return if $bad_component->{$_} }
 
-       $di->{path_a} = join('/', @a);
+       $di->{path_a} = join('/', @a) if @a;
        $di->{path_b} = join('/', @b);
 
        my $path = ++$self->{tot};
@@ -233,9 +233,8 @@ sub find_extract_diffs ($$$) {
        my $diffs = [];
        foreach my $smsg (@$msgs) {
                $ibx->smsg_mime($smsg) or next;
-               my $mime = delete $smsg->{mime};
-               msg_iter($mime, \&extract_diff,
-                               [$self, $diffs, $pre, $post, $ibx, $smsg]);
+               msg_iter(delete $smsg->{mime}, \&extract_diff,
+                               [$self, $diffs, $pre, $post, $ibx, $smsg], 1);
        }
        @$diffs ? $diffs : undef;
 }