git-svn: check error code of send_txstream
authorEric Wong <normalperson@yhbt.net>
Mon, 1 Sep 2008 02:45:07 +0000 (19:45 -0700)
committerEric Wong <normalperson@yhbt.net>
Fri, 5 Sep 2008 23:58:38 +0000 (16:58 -0700)
Not checking the error code of a function used to transform and
send data makes me nervous.  It currently returns "undef" on
success; so die if we get any result other than "undef" because
it's likely something went wrong somewhere.  I really wish this
function returned an MD5 like send_stream (or better yet, SHA1)
for verification.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
git-svn.perl

index fdf4e4a455cb80e2c145aad356638640491d6c8a..2d355c183439238cd3dccda49f01bae3c19f932f 100755 (executable)
@@ -3697,7 +3697,11 @@ sub chg_file {
        my $atd = $self->apply_textdelta($fbat, $exp_a, $pool);
        if (-s $fh_a) {
                my $txstream = SVN::TxDelta::new ($fh_a, $fh_b, $pool);
-               SVN::TxDelta::send_txstream($txstream, @$atd, $pool);
+               my $res = SVN::TxDelta::send_txstream($txstream, @$atd, $pool);
+               if (defined $res) {
+                       die "Unexpected result from send_txstream: $res\n",
+                           "(SVN::Core::VERSION: $SVN::Core::VERSION)\n";
+               }
        } else {
                my $got = SVN::TxDelta::send_stream($fh_b, @$atd, $pool);
                die "Checksum mismatch\nexpected: $exp_b\ngot: $got\n"