Re: [PATCH 1/6] revisions: refactor init_revisions and setup_revisions.

From: Pierre Habouzit <madcoder@debian.org>
Date: Jul 8, 2008, 12:48 UTC
Message-ID: <20080708124832.GH19202@artemis.madism.org>
In-Reply-To: <alpine.DEB.1.00.0807081339270.4319@eeepc-johanness>

On mar, jui 08, 2008 at 11:43:15 +0000, Johannes Schindelin wrote:

Show 20 quoted lines

> Hi,
> 
> On Tue, 8 Jul 2008, Pierre Habouzit wrote:
> 
> > On Tue, Jul 08, 2008 at 10:59:43AM +0000, Johannes Schindelin wrote:
> > 
> > > On Tue, 8 Jul 2008, Pierre Habouzit wrote:
> > > 
> > > > setup_revisions has been split in two: parse_revisions that does 
> > > > (almost) only argument parsing, to be more like what parse-options 
> > > > can do, and setup_revisions that does the rest.
> > > 
> > > I do not see the sense of this change, except
> > 
> > Well, it's required to remove "parse_revisions" at once if we one day 
> > reach the goal of having only parse-opt based parsers.
> 
> We can do that, then.
> 
> We do not need to do that, now.

  hmmm right, I may be able to cook a simpler series without that
indeed. Ill repost a new one with this idea.

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