El 1/12/2007, a las 15:07, Wincent Colaiuta escribió:
+# filters out prefixes which have special meaning to
list_and_choose()
+sub is_valid_prefix {
+ my $prefix = shift;
+ my $valid = (defined $prefix) &&
+ !($prefix =~ /[\s,]/) && # separators
+ !($prefix =~ /^-/) && # deselection
+ !($prefix =~ /^\d+/) && # selection
+ ($prefix ne '*'); # "all" wildcard
+}
Doh, that's supposed to be:
return (defined $prefix)...
Not:
my $valid = (defined $prefix)...
It actually works as is, but I had changed the "return" while working
on the patch (for debugging) and forgot to change it back afterwards.
And yes, I did proofread the patch before sending it. I just didn't
notice the first time around.
Cheers,
Wincent
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
