Thread (120 messages) 120 messages, 4 authors, 2016-06-16

Re: [PATCH v2 54/94] builtin/apply: make parse_chunk() return a negative integer on error

From: Christian Couder <hidden>
Date: 2016-06-16 02:19:48

On Mon, May 16, 2016 at 5:04 AM, Eric Sunshine [off-list ref] wrote:
On Wed, May 11, 2016 at 9:17 AM, Christian Couder
[off-list ref] wrote:
quoted
To libify `git apply` functionality we have to signal errors to the
caller instead of die()ing or exit()ing.

To do that in a compatible manner with the rest of the error handling
in builtin/apply.c, find_header() should return -1 instead of calling
die() or exit().
Why is this talking about making find_header() return -1? Didn't that
happen in the previous patch?
Yeah, it should be parse_chunk() not find_header().

This is fixed in my current branch.

Thanks,
Christian.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help