On Mon, Nov 05, 2007 at 09:48:19PM +0000, Junio C Hamano wrote:
quoted text
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>=20
> > After kicking this around a bit more on IRC, we had another idea.  Inst=
ead=20
quoted text
> > of introducing OPT_RECURSE(), do something like OPT__QUIET(), only this=
=20
quoted text
> > 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=
=20
quoted text
> > opinions.
> >
> > Hmm?
>=20
> As I am from "80s" ;-)

Hey grand'pa, how are your rheumatisms today :-P

quoted text
> I like the simpler "macro" one much better.  There aren't many things
> that can go wrong in the approach.

Okay, I suppose we will end up using the big macros then. Though I had a
look at diff_opt_parse and setup_revisions, the move isn't for tomorrow.

  The current code is heavy (at best), and has some logic that parseopt
is not yet ready to deal with (mainly the --not flags).

--=20
=C2=B7O=C2=B7  Pierre Habouzit
=C2=B7=C2=B7O                                                madcoder@debia=
n.org
OOO                                                http://www.madism.org