From @pmichaud

The rindex method added in r30858 has a bug​:

$ ./parrot perl6.pbc

say 85224.rindex('2');
Method 'reverse_index' not found for invocant of class 'Int'

Pm

From @moritz

Patrick R. Michaud (via RT) wrote​:

# New Ticket Created by Patrick R. Michaud
# Please include the string​: [perl #​58644]
# in the subject line of all future correspondence about this issue.
# <URL​: http://rt.perl.org/rt3/Ticket/Display.html?id=58644 >

The rindex method added in r30858 has a bug​:

$ ./parrot perl6.pbc

say 85224.rindex('2');
Method 'reverse_index' not found for invocant of class 'Int'

Added a (skipped) test to t/spec/S29-str/rindex.t

Moritz

--
Moritz Lenz
http://moritz.faui2k3.org/ | http://perl-6.de/

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

From @bacek

On Sun Sep 07 09​:32​:01 2008, pmichaud wrote​:

The rindex method added in r30858 has a bug​:

$ ./parrot perl6.pbc

say 85224.rindex('2');
Method 'reverse_index' not found for invocant of class 'Int'

Hello.

There is a patch for casting non-Strings to String. But I'm not quite
sure, that it is correct way...

--
Bacek

From @bacek

diff --git a/languages/perl6/src/builtins/any-str.pir b/languages/perl6/src/builtins/any-str.pir
index f5c4bf2..3e2a387 100644
--- a/languages/perl6/src/builtins/any-str.pir
+++ b/languages/perl6/src/builtins/any-str.pir
@@ -106,7 +106,16 @@ the size of that file down and to emphasize their generic,
     goto done
 
   substring_search:
-    pos = self.'reverse_index'(substring, pos)
+    $I0 = self.'isa'('String')
+    if $I0 goto self_string
+    $P0 = new 'String'
+    $S0 = self
+    $P0 = $S0
+    goto do_search
+  self_string:
+    $P0 = self
+  do_search:
+    pos = $P0.'reverse_index'(substring, pos)
     if pos < 0 goto fail
 
   done:

From @moritz

On Sun Sep 07 21​:40​:06 2008, bacek wrote​:

On Sun Sep 07 09​:32​:01 2008, pmichaud wrote​:

The rindex method added in r30858 has a bug​:

$ ./parrot perl6.pbc

say 85224.rindex('2');
Method 'reverse_index' not found for invocant of class 'Int'

Hello.

There is a patch for casting non-Strings to String. But I'm not quite
sure, that it is correct way...

Well, we need a cast somehow, I don't see why it should be wrong
(though my PIR fu is limited).

Applied as r30947, thank you.

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