On 20 September 2011 10:41, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Jay Soffian <jaysoffian@xxxxxxxxx> writes: > >>> I suspect that it would be more consistent with 6b44577 (mergetool: check >>> return value from read, 2011-07-01), which this patch is a follow-up to, >>> to do: >>> >>> read answer || return 1 >>> >>> here. >> >> Thanks, sorry I missed that. > > Thank _you_ for spotting these unchecked "read"s. Will queue. > -- > 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 > I got hit by the ctrl+c bug while using mergetool just the other day. Thanks for the fix :) -- 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