Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'is also' doesn't work on class Str (but everywhere else) #152

Closed
p6rt opened this issue Jul 4, 2008 · 5 comments
Closed

'is also' doesn't work on class Str (but everywhere else) #152

p6rt opened this issue Jul 4, 2008 · 5 comments
Labels

Comments

@p6rt
Copy link

p6rt commented Jul 4, 2008

Migrated from rt.perl.org#56602 (status was 'resolved')

Searchable as RT56602$

@p6rt
Copy link
Author

p6rt commented Jul 4, 2008

From @moritz

Rakudo r29047​:

../../parrot perl6.pbc -e 'class Int is also { method another_one { } }'
Class 'Int' already registered!

Classes are "open" by default in Perl 6, so the above should work.

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

@p6rt
Copy link
Author

p6rt commented Nov 23, 2008

From @moritz

On Fri Jul 04 01​:27​:01 2008, moritz@​casella.verplant.org wrote​:

Rakudo r29047​:

../../parrot perl6.pbc -e 'class Int is also { method another_one { } }'
Class 'Int' already registered!

Classes are "open" by default in Perl 6, so the above should work.

This works now, tests added in (pugs) r23064, t/spec/S12-class/open.t
(and added t/spectest.data in r33054). Very nice work, whoever did this ;-)

However the tests discovered that class Str can't be extended thusly
("Method 'mydouble' not found for invocant of class 'Perl6Str'", you get
that output if you run the test file directly instead of the fudged
version), so I'll leave this ticket open for now.

Cheers,
Moritz

@p6rt
Copy link
Author

p6rt commented Nov 23, 2008

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

@p6rt
Copy link
Author

p6rt commented Dec 5, 2008

From @jnthn

On Sun Nov 23 11​:31​:29 2008, moritz wrote​:

However the tests discovered that class Str can't be extended thusly
("Method 'mydouble' not found for invocant of class 'Perl6Str'", you get
that output if you run the test file directly instead of the fudged
version), so I'll leave this ticket open for now.

This is now fixed, and I unfudged the test.

Thanks!

Jonathan

@p6rt p6rt closed this as completed Dec 5, 2008
@p6rt
Copy link
Author

p6rt commented Dec 5, 2008

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

@p6rt p6rt added the Todo label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant