Hi, On Mon, 5 Nov 2007, Johannes Schindelin wrote: > The diff options should not need to be defined in every user of the > diffcore. This provides the framework: > > extern struct option *diff_options; > > struct option options[] = { > OPT_RECURSE(diff_options), > ... > OPT_END(), > }; After kicking this around a bit more on IRC, we had another idea. Instead of introducing OPT_RECURSE(), do something like OPT__QUIET(), only this time in diff.h: #define OPT__DIFF(opt) \ OPT_BOOLEAN('p', NULL, &opt.format_patch, "show a patch"), \ ... Pierre said this feels a bit "80s", so I'd like to hear other people's opinions. Hmm? Ciao, Dscho - 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