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

'proto' should imply 'multi' on all subsequent routines of that name in Rakudo #3143

Closed
p6rt opened this issue May 20, 2013 · 5 comments
Closed
Labels

Comments

@p6rt
Copy link

p6rt commented May 20, 2013

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

Searchable as RT118069$

@p6rt
Copy link
Author

p6rt commented May 20, 2013

From @masak

<dagurval> r​: proto sub a($) {*}; sub a(Int $x) { "yay" }
<camelia> rakudo 8d2ec9​: OUTPUT«===SORRY!===␤Redeclaration of routine a [...]
<dagurval> shouldn't that work? from syn​: ". A proto also adds an
implicit multi to all routines of the same short name within its
scope, unless they have an explicit modifier"
* masak submits rakudobug for that
<moritz> I think that's already (long) known
<masak> yes, but I don't find it in RT.
<masak> submitting Justin Case, and then we can merge if someone finds it.

@p6rt
Copy link
Author

p6rt commented Jun 19, 2015

From @usev6

Just a note​: There are tests for this in S06-multi/proto.t. Also two tests in S12-methods/multi.t are skipped for the same reason.

@p6rt
Copy link
Author

p6rt commented Jun 19, 2015

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

@p6rt
Copy link
Author

p6rt commented Oct 7, 2015

From @jnthn

On Mon May 20 10​:55​:06 2013, masak wrote​:

<dagurval> r​: proto sub a($) {*}; sub a(Int $x) { "yay" }
<camelia> rakudo 8d2ec9​: OUTPUT«===SORRY!===␤Redeclaration of routine
a [...]
<dagurval> shouldn't that work? from syn​: ". A proto also adds an
implicit multi to all routines of the same short name within its
scope, unless they have an explicit modifier"
* masak submits rakudobug for that
<moritz> I think that's already (long) known
<masak> yes, but I don't find it in RT.
<masak> submitting Justin Case, and then we can merge if someone finds
it.

After some discussion, we decided to remove this from the design​:

Raku/old-design-docs@8ee967d

I've also removed the fudged tests linked to this ticket.

@p6rt p6rt closed this as completed Oct 7, 2015
@p6rt
Copy link
Author

p6rt commented Oct 7, 2015

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

@p6rt p6rt added the Bug 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