On Tue, Aug 28, 2007 at 02:21:03AM -0400, Shawn O. Pearce wrote: > > + print $fi "M 100644 inline $fn\n"; > fast-import will also accept the shorter (perhaps more common) form: > > print $fi "M 644 inline $fn\n"; That is much more readable, and considering that this is meant to be an instructive example, I think it's worth changing. > > + print $fi "data $st[7]\n"; > This might be more clear as: > > print $fi "data $len\n"; Oops, yes; that was left from a previous iteration. I will incorporate both and re-send after 1.5.3; thanks for your comments. -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