Thread (7 messages) flat view 7 messages, 6 authors, 2016-06-15

Re: [PATCH] sha1_file.c: Avoid multiple calls to find_pack_entry().

From: Shawn O. Pearce <hidden>
Date: 2016-06-15 22:42:50

Simon 'corecode' Schubert [off-list ref] wrote:
Peter Eriksen wrote:
quoted
We used to call find_pack_entry() twice from read_sha1_file() in order
to avoid printing an error message, when the object did not exist.  This
is fixed by moving the call to error() to the only place it really
could be called.

Signed-off-by: Peter Eriksen <redacted>
I noticed this originally, Peter was so kind to come up with a patch.  
Reviewed and found +1, so:

Signed-off-by: Simon 'corecode' Schubert <redacted>
Wow.  That's ugly.  Thanks for finding and patching it.

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