savannah-hackers-public
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Savannah-hackers-public] Re: log_accum


From: Derek R. Price
Subject: [Savannah-hackers-public] Re: log_accum
Date: Fri, 17 Nov 2006 14:25:24 -0500
User-agent: Thunderbird 1.5.0.8 (Windows/20061025)

Sylvain Beucler wrote:
>>>  my @option_spec = ("config|c=s@",
>>>                "mail-to|m=s@",
>>> -              "tag|only-tag|r=s@",
>>> +              "only-tag|tag|r=s@",
>>>                "file-prefix|file-text|T=s", "user|u=s",
>> I don't see any good reason for this reordering, but I included it
>> anyhow.
> 
> That one is important (maybe I should have commented my changes :/),
> because GetOpt uses the first alternative in $config afaics. So in
> this case: $config{only-tag} (previously: $config{tag}).
> 
> &{$config{'only-tag'} won't be called if 'tag' is the first
> alternative. Since you already used $config{tag} to store the option
> values and $config{only-tags} as the callback, I just set 'only-tag'
> as the first alternative.

Ah, yes.  That sounds right.  Sorry for the confusion and thanks for the
refresher!  :)

Derek
-- 
Derek R. Price
CVS Solutions Architect
Get CVS support at Ximbiot <http://ximbiot.com>!
v: +1 248.835.1260
f: +1 248.835.1263
<mailto:address@hidden>





reply via email to

[Prev in Thread] Current Thread [Next in Thread]