On Fri, 19 Feb 2010, Karsten Br?ckelmann wrote:

On Thu, 2010-02-18 at 16:05 -0800, John Hardin wrote:
ERROR: LINT FAILED, suppressing output: rules/72_active.cf

WTF? That passed a full make+make test and masscheck here!

Funny. After a quick look at the blamed RE, trying to keep my eyes from bleeding -- yes, it indeed misses the closing parenthesis exactly there.

Write-only REs. Sorry. :)

Also, around the same lines:

 ( <-- HERE ?:randomly\s
   (?:selected|cho+sen|cho+sing|picked)|
   (?:selected|cho+sen|cho+sing|picked)\s [...]

Without real close investigation, that seems a little bit redundant.

I'll take a closer look, but I think it's an (A B)|(B C) situation and you're looking at B there.

--
 John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
 [email protected]    FALaholic #11174     pgpk -a [email protected]
 key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
  If guns kill people, then...
    -- pencils miss spel words.
    -- cars make people drive drunk.
    -- spoons make people fat.
-----------------------------------------------------------------------
 4 days until George Washington's 278th Birthday

Reply via email to