On 01/09/2013 03:59 AM, Mr Dash Four wrote:
> 
>>> Your patch had nothing to do with dashes in ipset names. It rather 
>>> removed
>>> the requirement for the ipset name to be preceded by '+'. 
>> Oh, yeah? From the patch I already attached in that email:
>>
>> -           fatal_error "Expected ipset name ($setname)" unless 
>> $setname =~ s/^\+// && $setname =~ /^(6_)?[a-zA-Z][-\w]*$/;
>> +           fatal_error "Expected ipset name ($setname)" unless 
>> $setname =~ /^(6_)?[a-zA-Z][\-\w]*$/;
>>
>> Pay close attention to the bit with "[\-\w]" at the end of the second 
>> line and then come back and tell me whether my patch "had nothing to 
>> do with dashes"?
> I just realised that "[\-\w]" and "[-\w]" are equivalents. Fair enough, 
> Tom, you have my apologies.
> 

Accepted.

-Tom
-- 
Tom Eastep        \ When I die, I want to go like my Grandfather who
Shoreline,         \ died peacefully in his sleep. Not screaming like
Washington, USA     \ all of the passengers in his car
http://shorewall.net \________________________________________________

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
and much more. Keep your Java skills current with LearnJavaNow -
200+ hours of step-by-step video tutorials by Java experts.
SALE $49.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122612 
_______________________________________________
Shorewall-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/shorewall-devel

Reply via email to