From: Jonathan Nieder Date: Tue, 27 Feb 2018 23:03:16 GMT Subject: Re: [PATCH v3 26/35] transport-helper: remove name parameter Message-ID: <20180227230316.GE174036@aiede.svl.corp.google.com> In-Reply-To: <20180207011312.189834-27-bmwill@google.com> Brandon Williams wrote: > Commit 266f1fdfa (transport-helper: be quiet on read errors from > helpers, 2013-06-21) removed a call to 'die()' which printed the name of > the remote helper passed in to the 'recvline_fh()' function using the > 'name' parameter. Once the call to 'die()' was removed the parameter > was no longer necessary but wasn't removed. Clean up 'recvline_fh()' > parameter list by removing the 'name' parameter. > > Signed-off-by: Brandon Williams > --- > transport-helper.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Nice. Reviewed-by: Jonathan Nieder