convert2RegExp missing case "}"

[erikvold]

Hey,

I think that the convert2RegExp function is missing the case "}".

Erik

[arantius]

Can you be a bit more specific ?

[erikvold]

The convert2regexp has a switch statement, the switch statement is missing the case statement for "}" because a "}" needs to become "}" as is aleady done with "{" already

Otherwise the pattern string "http://abc{3}" would become /^http://abc{3}$/

[erikvold]

hmmm /^http://abc{3}$/ still seems to work as a regular expression it seems, on FF3.6 at least, so maybe it's not an issue after all..

[erikvold]

It looks like "}" and "]" cases are treated similarly, so either we should handle the "}" case or not handle the "]" case to be consistent.

[arantius]

commit f115f8e