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

Implement 'is readonly' for private attributes in Rakudo #1440

Closed
p6rt opened this issue Dec 16, 2009 · 7 comments
Closed

Implement 'is readonly' for private attributes in Rakudo #1440

p6rt opened this issue Dec 16, 2009 · 7 comments
Labels

Comments

@p6rt
Copy link

p6rt commented Dec 16, 2009

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

Searchable as RT71356$

@p6rt
Copy link
Author

p6rt commented Dec 16, 2009

From @masak

<masak> rakudo​: class A { has $!b is readonly = "foo" }; A.new
<p6eval> rakudo 55f51d​: ( no output )
<masak> locally, "No applicable candidates found to dispatch to for
'trait_mod​:is'"
<masak> should that work?
<jnthn> masak​: Probably...
* masak submits rakudobug
<masak> at least the warning is LTA-ly suspicious.
<jnthn> Well, it's not far off the mark.
<masak> it makes it sound as if 'is readonly' is not implemented...
<jnthn> Right, which in that case it evidently isn't. :-)
<masak> so, to summarize​: the bug is that 'is readonly' isn't implemented.
<masak> we also don't really know whether it should work,
semantically, to assign to a readonly variable during object
construction.

@p6rt
Copy link
Author

p6rt commented Dec 10, 2012

From @coke

08​:44 < timotimo> triage mode engaged​:
  https://rt-archive.perl.org/perl6/Public/Bug/Display.html?id=71356 -
  this bug seems resolved. can be closed.

Marking testneeded.

--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Dec 10, 2012

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

@p6rt
Copy link
Author

p6rt commented Dec 11, 2012

From @timo

A test case for this bug was added to roast in this commit​:

Raku/roast@1fa9ddf
8b67d

@p6rt
Copy link
Author

p6rt commented Oct 28, 2015

From @stmuk

On Tue Dec 11 11​:27​:59 2012, timo wrote​:

A test case for this bug was added to roast in this commit​:

Raku/roast@1fa9ddf

I notice this warns with "useless use of 'is readonly' on $!attr"
and the test is still #?rakudo todo 'readonly attributes'

@p6rt
Copy link
Author

p6rt commented Jul 6, 2016

From @zoffixznet

As was discussed at http://irclog.perlgeek.de/perl6/2016-07-06#i_12795183 this feature/tests are no longer relevant in the current Rakudo implementation.

@p6rt
Copy link
Author

p6rt commented Jul 6, 2016

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

@p6rt p6rt closed this as completed Jul 6, 2016
@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