🚀 go-pugleaf

Ok, here's an obscure one for y'all ...

I put a line in my NNTP killfile as follows:
    Subject: *[ _-]LOOT[ _-]*rail*
wanting to catch "lootrail", "loot-rail", "loot_rail" and "loot rail"
and similar combinations (if you read misc.transport.urban-transit
you'll understand).

But it also catches a whole lot more.  Setting nntp trace to 3 shows
me that it also kills stuff like:

    Subject: Re: Poly marriage contract
    Subject: Re: More Junk Email
    Subject: Re: "Name of a dog! Rover

Looking at the code for wildmat(), I guess it is getting confused about
the hyphen and imagining it is part of a character range.  A hyphen as
first or last in the string ought to be treated as a literal, but
wildmat doesn't conform to this convention.

The line in wildmat.c, case '[', that says

    if(*p == '-' ? *s <= *++p && *s >= last : *s == *p)
	matched = TRUE;

is the one in error.  I was going to suggest a correction but I'm having
trouble in coming up with one as efficient as KA9Q standard !  It's not
too hard, obviously, to do by brute force of code, but I'd like to keep
the memory requirements down in any fix ...

Nick