Re: [PATCH] parseopt: introduce OPT_RECURSE to specify shared options

From: Pierre Habouzit <madcoder@debian.org>
Date: Nov 5, 2007, 13:53 UTC
Message-ID: <20071105135319.GA6205@artemis.corp>
In-Reply-To: <20071105123923.GC25574@artemis.corp>

On Mon, Nov 05, 2007 at 12:39:23PM +0000, Pierre Habouzit wrote:

>   I like the kind of code that I allow to write better (I tend to
> dislike big fat global variables), though it's obvious that Johannes
> patch is a lot simpler and I like that.

  We discussed it further, and what came out is that instead of
supporting quite complicated recursion mechanisms (or even a non so
complicated one), we can just define an
OPT__DIFFOPTIONS(diffopts)/OPT__REVOPTIONS(rev) macro that would inline
the needed options.

  That's an idea I had but dismissed. Though, maybe it's not _that_
ugly, is clearly simpler, and one can argue that it's in the logical
continuation of OPT__QUIET and friends. What do you think ?

  If it's the road we decide to take, then my documentation patch (2/4),
the parseopt fix (my 1/4 or johannes 1/3) and Johannes usage generator
enhancement (his 3/3) are still to be taken.

-- 
·O·  Pierre Habouzit
··O                                                madcoder@debian.org
OOO                                                http://www.madism.org