On Tue, Sep 28, 2010 at 06:35:40PM +0400, Kirill Smelkov wrote: > > > t/t4042-diff-textconv-caching.sh | 4 ++-- > > > > Why are we touching t4042 at all in this series? We are not actually > > adding any tests to it, AFAICT. > > Because we want to catch potential wrong textconv invocation on non > "bin: " files there too? But we don't actually add any tests that display the problem there, do we? And even if we wanted to test the diff implementation, wouldn't t4030 be the write place to do that? t4042 is specifically about textconv caching. -Peff -- 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