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

Re: [PATCH 2/2] Documentation/remote-helpers: Fix some typos

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:48:33

Ramkumar Ramachandra [off-list ref] writes:
quoted hunk
@@ -128,9 +131,9 @@ Supported if the helper has the "import" capability.
 Supported if the helper has the "connect" capability.

 If a fatal error occurs, the program writes the error message to
-stderr and exits. The caller should expect that a suitable error
-message has been printed if the child closes the connection without
-completing a valid response for the current command.
+stderr and exits. The caller should expect a suitable error
+if the child closes the connection without
+giving a valid response for the current command.
This not just fixes typo but changes what is being said, doesn't it?  The
point of the original about the "error" is that the caller does not have
to issue a redundant error message when the helper closes the connection
without reporting the status back, but that information which probably is
critical when maintaining the caller (and writing a new callee) is lost.

Other parts looked Ok to me.

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