From: Pierre Habouzit Date: Sun, 14 Oct 2007 07:02:19 GMT Subject: Re: [PATCH] Add a simple option parser. Message-ID: <20071014070219.GA1198@artemis.corp> In-Reply-To: <20071013221450.GC2875@steel.home> On Sat, Oct 13, 2007 at 10:14:50PM +0000, Alex Riesen wrote: > Pierre Habouzit, Sat, Oct 13, 2007 22:54:04 +0200: > > On Sat, Oct 13, 2007 at 07:16:55PM +0000, Alex Riesen wrote: > > > Pierre Habouzit, Sat, Oct 13, 2007 15:29:03 +0200: > > > BTW, if you just printed the usage message out (it is about usage of a > > > program, isn't it?) and called exit() everyone would be just as happy. > > > And you wouldn't have to include strbuf (it is the only use of it), > > > less code, too. It'd make simplier to stea^Wcopy your implementation, > > > which I like :) > > > > the reason is that usage() is a wrapper around a callback, and I > > suppose it's used by some GUI's or anything like that. > > It is not. Not yet. What could they use a usage text for? > Besides, you could just export the callback (call_usage_callback or > something) from usage.c and call it. Okay makes sense. > > FWIW you can rework the .c like this: > > on top of yours: Added (reworked a bit for the current state of parse_options), and pushed. -- ·O· Pierre Habouzit ··O madcoder@debian.org OOO http://www.madism.org