On Tue, Jun 28, 2016 at 11:39 PM, Junio C Hamano <gits...@pobox.com> wrote:
> Christian Couder <christian.cou...@gmail.com> writes:
>
>> @@ -98,8 +97,7 @@ int write_or_whine_pipe(int fd, const void *buf, size_t
>> count, const char *msg)
>> int write_or_whine(int fd, const void *buf, size_t count, const char *msg)
>> {
>> if (write_in_full(fd, buf, count) < 0) {
>> - fprintf(stderr, "%s: write error (%s)\n",
>> - msg, strerror(errno));
>> + warning("%s: write error (%s)\n", msg, strerror(errno));
>> return 0;
>> }
>
> I do not think you call write_or_whine() at all. As another topic
> in flight removes the last caller of this function, this hunk is
> very much unwelcome. The only effect of it is to force me resolve
> unnecessary merge conflicts.
Ok, sorry about that. I will remove the hunk.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
