since my attachment probably prevented the mail reaching the d-www list,
I'll CC this one (see bug #563030 for missed mails and patch).
On Sat, 25 Jun 2011 10:50:15 -0700
Don Armstrong <firstname.lastname@example.org> wrote:
> > But since I believe nobody actually does that (easier to simply
> > strip the #)
> I actually do, which is why it works.
Please pardon me assuming so then.
> Awesome. However, the regex for this is needlessly complicated [and
> because of that, it probably is incorrect.]
> This should really just be:
> var url_test = /^https?://.+\.debian.org/Bugs/\s*\#(\d+)\s*$/;
> or similar. [Someone who actually writes JS can probably make the
> double testing slightly more elegant.]
Yes, I am not attached to that regex, but don't feel that
comfortable writing one (it would probably break on something ;)
Why I chose that regex, is that it is used by the MooTools project (a
anything URI-related there. It is also really universal.
It has seen no changes since January 2010 (according to blame), so I
would believe it is well tested.
For verification, it can be found on github, Line 42, (sorry about the