From @pmichaud

On Sat, Sep 20, 2008 at 12​:33​:27AM +1100, Илья wrote​:

For example​:
split /''/, $string
splits between characters.

this do not work properly in Rakudo​:
say (split /''/, "foo").perl; # ["foo"]

Currently the split function (note function, not method) is in
src/classes/Str.pir, and it appears to be coercing its arguments
into strings.

This function needs to be moved to src/builtins/any-str.pir,
and updated to accept pmc arguments. Perhaps something like​:

  .namespace ['']
  .sub 'split' :multi(_, _)
  .param pmc sep
  .param pmc target
  .return target.'split'(sep)
  .end

Pm

From @moritz

On Fri Sep 19 06​:49​:37 2008, pmichaud wrote​:

On Sat, Sep 20, 2008 at 12​:33​:27AM +1100, Илья wrote​:

For example​:
split /''/, $string
splits between characters.

this do not work properly in Rakudo​:
say (split /''/, "foo").perl; # ["foo"]

Currently the split function (note function, not method) is in
src/classes/Str.pir, and it appears to be coercing its arguments
into strings.

This function needs to be moved to src/builtins/any-str.pir,
and updated to accept pmc arguments. Perhaps something like​:

\.namespace \[''\]
\.sub 'split' :multi\(\_, \_\)
    \.param pmc sep
    \.param pmc target
    \.return target\.'split'\(sep\)
\.end

Pm

It has been moved, but it doesn't solve all problems. To be more specific​:

split("4", "345").perl.say
["3", "5"]
split(4, "345").perl.say
["3", "5"]
split(4, 345).perl.say
No applicable methods.
split(/b/, 'abc').perl.say
["a", "c"]

So it seems the only thing left to do is make split with non-regex and
non-string delimiter work on non-strings.

(Other TODO items are the behaviour with regexes that produce zero-width
matches, but that's for another ticket, I think).

Moritz

The RT System itself - Status changed from 'new' to 'open'

From @moritz

Fixed in r31333.

@moritz - Status changed from 'open' to 'resolved'