Re: [PATCH 6/9] streaming_write_entry: propagate streaming errors

2013-03-25 Thread Jonathan Nieder
Jeff King wrote: > When we are streaming an index blob to disk, we store the > error from stream_blob_to_fd in the "result" variable, and > then immediately overwrite that with the return value of > "close". Good catch. [...] > --- a/entry.c > +++ b/entry.c > @@ -126,8 +126,10 @@ static int stre

Re: [PATCH 6/9] streaming_write_entry: propagate streaming errors

2013-03-25 Thread Jeff King
On Mon, Mar 25, 2013 at 05:35:51PM -0400, Eric Sunshine wrote: > On Mon, Mar 25, 2013 at 4:22 PM, Jeff King wrote: > > diff --git a/entry.c b/entry.c > > index 17a6bcc..002b2f2 100644 > > --- a/entry.c > > +++ b/entry.c > > @@ -126,8 +126,10 @@ static int streaming_write_entry(struct cache_entry

Re: [PATCH 6/9] streaming_write_entry: propagate streaming errors

2013-03-25 Thread Eric Sunshine
On Mon, Mar 25, 2013 at 4:22 PM, Jeff King wrote: > diff --git a/entry.c b/entry.c > index 17a6bcc..002b2f2 100644 > --- a/entry.c > +++ b/entry.c > @@ -126,8 +126,10 @@ static int streaming_write_entry(struct cache_entry *ce, > char *path, > fd = open_output_fd(path, ce, to_tempfile); >