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

'where' clauses on attributes do nothing in Rakudo #3415

Closed
p6rt opened this issue Jun 16, 2014 · 5 comments
Closed

'where' clauses on attributes do nothing in Rakudo #3415

p6rt opened this issue Jun 16, 2014 · 5 comments
Labels
NYI Features not yet implemented

Comments

@p6rt
Copy link

p6rt commented Jun 16, 2014

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

Searchable as RT122109$

@p6rt
Copy link
Author

p6rt commented Jun 16, 2014

From @masak

<masak> r​: class Conf { has @​.boxes where { all(@​$_) >= 0 } }; my $c =
Conf.new(​:boxes[ 2, 2, -1 ]); say $c.perl
<camelia> rakudo-{parrot,jvm,moar} f1b05d​: OUTPUT«Conf.new(boxes =>
Array.new(2, 2, -1))␤»
<masak> should this be allowed?
<moritz> nope
<Ven> masak​: It doesn't even look like the `where` is evaluated
<masak> r​: class C { has $.x where * > 0 }; say C.new(​:x(-42)).x
<camelia> rakudo-{parrot,jvm,moar} f1b05d​: OUTPUT«-42␤»
* masak submits rakudobug
<masak> niecza​: class C { has $.x where * > 0 }; say C.new(​:x(-42)).x
<camelia> niecza v24-109-g48a8de3​:
OUTPUT«===SORRY!===␤␤Postconstraints, and shapes on variable
declarators NYI [...]»
<masak> at least Niecza knows it can't.

@p6rt
Copy link
Author

p6rt commented Oct 20, 2014

From @usev6

I added a fudged (skip) test to S12-class/attributes.t with the following commit​: Raku/roast@cc6dfe8bd8

@p6rt
Copy link
Author

p6rt commented Oct 20, 2014

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

@p6rt
Copy link
Author

p6rt commented Apr 12, 2015

From @jnthn

On Mon Oct 20 10​:57​:35 2014, bartolin@​gmx.de wrote​:

I added a fudged (skip) test to S12-class/attributes.t with the
following commit​: Raku/roast@cc6dfe8bd8

I've done the work to make it pass, and un-fudged it.

@p6rt p6rt closed this as completed Apr 12, 2015
@p6rt
Copy link
Author

p6rt commented Apr 12, 2015

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

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

No branches or pull requests

1 participant