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

Apparently wrong precedence between assignment and infix:<(|)> in Rakudo #2998

Closed
p6rt opened this issue Dec 15, 2012 · 8 comments
Closed

Apparently wrong precedence between assignment and infix:<(|)> in Rakudo #2998

p6rt opened this issue Dec 15, 2012 · 8 comments

Comments

@p6rt
Copy link

p6rt commented Dec 15, 2012

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

Searchable as RT116100$

@p6rt
Copy link
Author

p6rt commented Dec 15, 2012

From @masak

<masak> r​: my $s = set; my $e = 5; $s = $s (|) $e; say $s.perl
<p6eval> rakudo cf2761​: OUTPUT«set()␤»
<masak> huh!
<masak> not how I expect (|) to work... :/
<masak> r​: my $s = set; my $e = 5; say ($s (|) $e).perl
<p6eval> rakudo cf2761​: OUTPUT«set("5")␤»
<masak> something is wrong when doing both '=' and '(|)'
* masak submits rakudobug
<jnthn> masak​: iirc, colomon++ mentioned he copied the precedence from STD
<jnthn> masak​: But it's suspect.
<jnthn> Or something like...
<colomon> Not from STD.
<jnthn> Ah, where from?
<colomon> I think that was the precedence TimToady had on it.
<jnthn> OK
* colomon wonders were the original Set implementation is on his hard drive.
<jnthn> Seems a bit surprising to me.
<colomon> definitely feels wrong to me after having tried it.
<jnthn> yeah

@p6rt
Copy link
Author

p6rt commented Jan 11, 2013

From @jnthn

On Sat Dec 15 09​:46​:04 2012, masak wrote​:

<masak> r​: my $s = set; my $e = 5; $s = $s (|) $e; say $s.perl
<p6eval> rakudo cf2761​: OUTPUT«set()␤»
<masak> huh!
<masak> not how I expect (|) to work... :/
<masak> r​: my $s = set; my $e = 5; say ($s (|) $e).perl
<p6eval> rakudo cf2761​: OUTPUT«set("5")␤»
<masak> something is wrong when doing both '=' and '(|)'
* masak submits rakudobug
<jnthn> masak​: iirc, colomon++ mentioned he copied the precedence from STD
<jnthn> masak​: But it's suspect.
<jnthn> Or something like...
<colomon> Not from STD.
<jnthn> Ah, where from?
<colomon> I think that was the precedence TimToady had on it.
<jnthn> OK
* colomon wonders were the original Set implementation is on his hard
drive.
<jnthn> Seems a bit surprising to me.
<colomon> definitely feels wrong to me after having tried it.
<jnthn> yeah

The spec and STD got its precedence for these updated, and Rakudo has
now followed suit, putting them at junctive precedences.

Tagging testneeded.

/jnthn

@p6rt
Copy link
Author

p6rt commented Jan 11, 2013

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Jan 28, 2014

From @peschwa

Am Fr 11. Jan 2013, 13​:56​:40, jnthn@​jnthn.net schrieb​:

On Sat Dec 15 09​:46​:04 2012, masak wrote​:

<masak> r​: my $s = set; my $e = 5; $s = $s (|) $e; say $s.perl
<p6eval> rakudo cf2761​: OUTPUT«set()␤»
<masak> huh!
<masak> not how I expect (|) to work... :/
<masak> r​: my $s = set; my $e = 5; say ($s (|) $e).perl
<p6eval> rakudo cf2761​: OUTPUT«set("5")␤»
<masak> something is wrong when doing both '=' and '(|)'
* masak submits rakudobug
<jnthn> masak​: iirc, colomon++ mentioned he copied the precedence from STD
<jnthn> masak​: But it's suspect.
<jnthn> Or something like...
<colomon> Not from STD.
<jnthn> Ah, where from?
<colomon> I think that was the precedence TimToady had on it.
<jnthn> OK
* colomon wonders were the original Set implementation is on his hard
drive.
<jnthn> Seems a bit surprising to me.
<colomon> definitely feels wrong to me after having tried it.
<jnthn> yeah

The spec and STD got its precedence for these updated, and Rakudo has
now followed suit, putting them at junctive precedences.

Tagging testneeded.

/jnthn

Added test to roast in commit db7936e as per the example above.

@p6rt
Copy link
Author

p6rt commented Feb 20, 2014

From @coke

jnthn++ added a passing test to roast; closing ticket.

--
Will "Coke" Coleda

1 similar comment
@p6rt
Copy link
Author

p6rt commented Feb 20, 2014

From @coke

jnthn++ added a passing test to roast; closing ticket.

--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Feb 20, 2014

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

@p6rt p6rt closed this as completed Feb 20, 2014
@p6rt
Copy link
Author

p6rt commented Feb 20, 2014

From @coke

On Thu Feb 20 07​:54​:07 2014, coke wrote​:

jnthn++ added a passing test to roast; closing ticket.

Whoops, was peschwa++

--
Will "Coke" Coleda

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