Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

[perl #41323] [CAGE] convert C<theINTERP> to C<interp>

4 views
Skip to first unread message

Jerry Gay

unread,
Jan 23, 2007, 12:49:59 PM1/23/07
to bugs-bi...@rt.perl.org
# New Ticket Created by Jerry Gay
# Please include the string: [perl #41323]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=41323 >


in november, there was a big cleanup of the different spellings of
C<interp>, hovewer, anything labelled C<theINTERP> was missed. all
references should be modified to C<interp>.
~jerry

Nuno Carvalho via RT

unread,
Jan 24, 2007, 12:02:20 PM1/24/07
to perl6-i...@perl.org
Hi all,

On Tue Jan 23 09:49:57 2007, particle wrote:
> in november, there was a big cleanup of the different spellings of
> C<interp>, hovewer, anything labelled C<theINTERP> was missed. all
> references should be modified to C<interp>.
> ~jerry

Here's a patch to replace all C<theINTERP>. I tested this on a x86
linux:

$ make test # before changes
(...)
All tests successful, 10 tests and 598 subtests skipped.
Files=268, Tests=6844, 606 wallclock secs (277.52 cusr + 78.61 csys =
356.13 CPU)

$ make test # after changes
(...)
All tests successful, 10 tests and 598 subtests skipped.
Files=268, Tests=6844, 579 wallclock secs (277.63 cusr + 77.68 csys =
355.31 CPU)

./smash

interp.patch

Paul Cochrane

unread,
Jan 25, 2007, 2:19:15 AM1/25/07
to parrotbug...@parrotcode.org

Patch applies cleanly and tests pass for me also on linux x86.
Applied as r16789. Thanks for the hard work!

Paul

Paul Cochrane

unread,
Jan 25, 2007, 3:46:20 AM1/25/07
to parrot-...@perl.org

This patch also affects win32. Could someone confirm that everything
is ok there and then I'll close the ticket.

Thanks!

Paul

Jerry Gay via RT

unread,
Jan 25, 2007, 10:35:22 AM1/25/07
to perl6-i...@perl.org
works great here on win32--ticket closed.
~jerry
0 new messages