Welcome! Please see the About page for a little more info on how this works.

0 votes
in ClojureScript by

Current behaviour:

(re-matches #"(a|aa)" "aa") => nil

Expected:

(re-matches #"(a|aa)" "aa") => ["aa" "aa"]

JVM version works as expected, only CLJS is affected

6 Answers

0 votes
by

Comment made by: dnolen

This is the kind of ticket that tends to break existing code. We should get some people who are interested in this ticket to actually try it out.

0 votes
by

Comment made by: mfikes

FWIW, I gave cljs-1458-re-matches-might-give-false-negative.patch a try in bootstrapped ClojureScript and it is working fine there (each of the additional unit tests produce the expected results in bootstrapped).

0 votes
by

Comment made by: mfikes

Patch no longer applies.

0 votes
by

Comment made by: tonsky

New patch

0 votes
by

Comment made by: mfikes

CLJS-1458-re-matches-might-give-false-negative-2.patch added to Patch Tender (i)

0 votes
by
Reference: https://clojure.atlassian.net/browse/CLJS-1458 (reported by tonsky)
...