Junio C Hamano <junkio@cox.net> writes: > Linus Torvalds <torvalds@osdl.org> writes: > >> Umm.. Why not just say "if the line after the 'diff' line doesn't start >> with '@@ ', then it's a binary diff"? > > Thanks, that makes sense. Maybe not; parse_single_patch would have eaten '@@ -' lines so in practice that amounts to not checking anything at all. Which is not a bad idea, though... That means we would treat all empty patches as binary diffs, which is certainly simple, but is it safe enough? - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Sat Nov 19 06:13:45 2005
This archive was generated by hypermail 2.1.8 : 2005-11-19 06:13:51 EST