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
No warning when op with same name as class accidentally used #1892
Comments
From publiustemp-perl6internals2@yahoo.comThis code: class x { Produces this output: Method 'y' not found for invocant of class 'Failure' It works if I put the "x.new" code in another file and "use x". It also works if I stay in the same file and I assign x.new to a variable and call .y on the variable. my $x = x.new; It fails on both master and ng, versions 7914ca3aa2f17ced09ff0707700e638d77cd5a1f and 2db8b5e6e8a5ea1559bf2668866e12476cf416fc respectively. Cheers, |
From @masakOvid (>):
This is a parsing issue. (Don't ask me how I know.) Put a semicolon after |
The RT System itself - Status changed from 'new' to 'open' |
From @bbkr[13:24] <moritz_> bbkr: spec says that after a } either a newline or a ; is # this code parses on Rakudo # but it's not valid according to STD # expected |
From @bbkr[14:40] <pmichaud> 11:19 <bbkr> rakudo: class X { has Int $.id; method perl6 -e 'class ABC { } ABC.new' Indeed. Still 'class X { } X.new' should not parse as X operator usage Or there should be more awesome message about X,Q,Z operators misuse. |
@bbkr - Status changed from 'new' to 'open' |
@bbkr - Status changed from 'open' to 'new' |
From @bbkrSimiliar issue discussed here http://rt.perl.org/rt3/Ticket/Display.html?id=76236 maybe those tickets can be merged? |
@coke - Status changed from 'new' to 'open' |
From @diakopterOn Mon Jun 28 06:30:29 2010, bbkr wrote:
this has about 2-3 other later duplicates. will maybe find/merge them |
From @labsterTL;DR testneeded on the first half, second half notabug. These two tickets should not have been merged, apparently. Because the original ticket, #72338: labster% cat foo.pl class x { The above can presumably be closed with a test. But on the second issue, #72636: However: |
1 similar comment
From @labsterTL;DR testneeded on the first half, second half notabug. These two tickets should not have been merged, apparently. Because the original ticket, #72338: labster% cat foo.pl class x { The above can presumably be closed with a test. But on the second issue, #72636: However: |
From @usev6On Tue Jun 16 22:19:50 2015, labster wrote:
There is already a (passing) test for the first half (RT #72338) in S12-class/basic.t. I'm going ahead and close this ticket as 'resolved'. Please re-open if you disagree with the classification 'notabug' for the second part. |
@usev6 - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#76236 (status was 'resolved')
Searchable as RT76236$
The text was updated successfully, but these errors were encountered: