Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> writes: > On Mon, 23 Jun 2008, Pierre Habouzit wrote: >> >> Let's see if I can catch it elegantly. > > No. > > Look at builtin-blame.c. > > THEN get back to me. > > Trust me, you need what I wrote. Something that parses all the options in > one go, and ignores the ones it cannot parse, because a TOTALLY DIFFERENT > function than the caller will call it! I do not think you two are saying vastly different things. The series Pierre has just posted is prerequisite for teaching parse_options() to queue the unknown ones (besides allowing a finer grained stepwise parsing), so that the caller like cmd_blame() can call revision parser with the remainder. -- 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