Ramkumar Ramachandra <artagnon@xxxxxxxxx> writes: > @@ -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. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html