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

can't add multi methods by monkey typing #1792

Closed
p6rt opened this issue May 29, 2010 · 6 comments
Closed

can't add multi methods by monkey typing #1792

p6rt opened this issue May 29, 2010 · 6 comments

Comments

@p6rt
Copy link

p6rt commented May 29, 2010

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

Searchable as RT75432$

@p6rt
Copy link
Author

p6rt commented May 29, 2010

From @moritz

16​:27 <@​moritz_> rakudo​: class A { multi method a() { }; multi method
a() { } }
16​:27 < p6eval> rakudo bbb336​: ( no output )
16​:27 <@​moritz_> rakudo​: class A { multi method a() { }}; use
MONKEY_TYPING;
  augment class A { multi method a() { } }
16​:27 < p6eval> rakudo bbb336​: OUTPUT«A method named 'a' already exists in
  class 'A'. It may have been supplied by a role.â�¤current
instr.​:
  'perl6;ClassHOW;add_method' pc 4969
  (src/metamodel/ClassHOW.pir​:502)â�¤Â»
16​:27 < Su-Shee> and has something which starts to resemble
"fetchrow_hashref"
16​:27 <@​moritz_> that's a bug, right?
16​:28 <@​jnthn> moritz_​: Looks like it.
16​:28 <@​moritz_> any masaks around?
16​:28 <@​moritz_> looks like not
16​:28 <@​jnthn> oh noes, how will we get an RT ticket filed?
16​:28 < Su-Shee> moritz_​: well write a FakeMasak then!

@p6rt
Copy link
Author

p6rt commented Jul 29, 2010

@coke - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Oct 5, 2011

From @coke

On Sat May 29 07​:30​:06 2010, moritz wrote​:

16​:27 <@​moritz_> rakudo​: class A { multi method a() { }; multi method
a() { } }
16​:27 < p6eval> rakudo bbb336​: ( no output )
16​:27 <@​moritz_> rakudo​: class A { multi method a() { }}; use
MONKEY_TYPING;
augment class A { multi method a() { } }
16​:27 < p6eval> rakudo bbb336​: OUTPUT«A method named 'a' already exists in
class 'A'. It may have been supplied by a role.�current
instr.​:
'perl6;ClassHOW;add_method' pc 4969
(src/metamodel/ClassHOW.pir​:502)â�¤Â»

Now says​:

20​:23 < [Coke]> rakudo​: class A { multi method a() { }}; use MONKEY_TYPING;
  augment class A { multi method a() { } }
20​:24 <+p6eval> rakudo 545638​: ( no output )

16​:27 < Su-Shee> and has something which starts to resemble
"fetchrow_hashref"
16​:27 <@​moritz_> that's a bug, right?
16​:28 <@​jnthn> moritz_​: Looks like it.
16​:28 <@​moritz_> any masaks around?
16​:28 <@​moritz_> looks like not
16​:28 <@​jnthn> oh noes, how will we get an RT ticket filed?
16​:28 < Su-Shee> moritz_​: well write a FakeMasak then!

Closable with tests.
--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Nov 15, 2011

From @tadzik

On Tue Oct 04 17​:24​:23 2011, coke wrote​:

Closable with tests.

Tests in 43de0a9, closable.

@p6rt
Copy link
Author

p6rt commented Nov 15, 2011

From @masak

On Tue Nov 15 08​:56​:58 2011, tadzik wrote​:

On Tue Oct 04 17​:24​:23 2011, coke wrote​:

Closable with tests.

Tests in 43de0a9, closable.

Thanks; resolved.

@p6rt
Copy link
Author

p6rt commented Nov 15, 2011

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

@p6rt p6rt closed this as completed Nov 15, 2011
@p6rt p6rt added the testneeded label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant