summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 8598db9)
raw | patch | inline | side by side (parent: 8598db9)
author | Eric Wong <normalperson@yhbt.net> | |
Mon, 1 Sep 2008 02:45:07 +0000 (19:45 -0700) | ||
committer | Eric 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>
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 | patch | blob | history |
diff --git a/git-svn.perl b/git-svn.perl
index fdf4e4a455cb80e2c145aad356638640491d6c8a..2d355c183439238cd3dccda49f01bae3c19f932f 100755 (executable)
--- a/git-svn.perl
+++ b/git-svn.perl
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"