Skip Menu | You are currently an anonymous guest. | Login | Return to Main | About rt.cpan.org
 

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.

X Report information
Id: 8733
Status: resolved
Left: 0 min
Priority: 0/0
Queue: Regexp-Optimizer

Owner: DANKOGAI <DANKOGAI [...] cpan.org>
Requestors: thomaspazhartman [...] fastmail.fm
Cc:
AdminCc:

Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



X History Display mode: Brief headersFull headers
#   Fri Dec 03 14:27:37 2004 guest - Ticket created  
Subject: seems to be a typo
[text/plain 387b]
http://search.cpan.org/~dankogai/Regexp-Optimizer-0.14/lib/Regexp/List.pm

has line
<code>
# $re is now qr/foo(?:[bx]ar|zap)/
</code>
then later in the documentation

<code>
"And the final result is reached.

foo(?:[bx]ar|zap?)
</code>

which has an extra ? after zap.

I think the first line is correct, but at any rate these should agree with each other.

Hope this helps!

thomas.

#   Sun Dec 05 08:35:49 2004 DANKOGAI - Taken  
#   Sun Dec 05 11:05:35 2004 DANKOGAI - Correspondence added  
[text/plain 597b]
[guest - Fri Dec 3 14:27:37 2004]:
> http://search.cpan.org/~dankogai/Regexp-Optimizer-
> 0.14/lib/Regexp/List.pm
>
> has line
> <code>
> # $re is now qr/foo(?:[bx]ar|zap)/
> </code>
> then later in the documentation
>
> <code>
> "And the final result is reached.
>
> foo(?:[bx]ar|zap?)
> </code>
>
> which has an extra ? after zap.

That was okay because the first example did not include "fooza".

> I think the first line is correct, but at any rate these should agree
> with each other.

I revised the pod so it is now less confusing. Thanks for your input.

Dan the Mantainer Thereof

#   Sun Dec 05 11:05:36 2004 DANKOGAI - Status changed from 'new' to 'resolved'