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

Segfault for a signature match with a native type and an 'as' cast in Rakudo #3173

Closed
p6rt opened this issue Jun 21, 2013 · 7 comments
Closed

Comments

@p6rt
Copy link

p6rt commented Jun 21, 2013

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

Searchable as RT118581$

@p6rt
Copy link
Author

p6rt commented Jun 21, 2013

From @masak

<jnthn> r​: say \(1) ~~ :(int $x as Str)
<camelia> rakudo 51d024​: OUTPUT«(signal SEGV)»
<masak> O.O
* masak submits rakudobug
<jnthn> Found while porting the code for the JVM :)

@p6rt
Copy link
Author

p6rt commented Oct 16, 2014

From @usev6

This no longer segfaults but gives False (on Moar, Parrot and JVM)​:

$ perl6 -e 'say \(1) ~~ :(int $x as Str)'
False

Is that smart match supposed to return True? (For the tests it would be nice to check for the correct result and not only whether it lives_ok.)

1 similar comment
@p6rt
Copy link
Author

p6rt commented Oct 16, 2014

From @usev6

This no longer segfaults but gives False (on Moar, Parrot and JVM)​:

$ perl6 -e 'say \(1) ~~ :(int $x as Str)'
False

Is that smart match supposed to return True? (For the tests it would be nice to check for the correct result and not only whether it lives_ok.)

@p6rt
Copy link
Author

p6rt commented Oct 16, 2014

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

@p6rt
Copy link
Author

p6rt commented Oct 16, 2014

From @masak

Resolving this ticket because Rakudo no longer segfaults.

See [perl #​122990] for the unexpected behavior it revealed underneath.

@p6rt
Copy link
Author

p6rt commented Oct 16, 2014

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

@p6rt p6rt closed this as completed Oct 16, 2014
@p6rt
Copy link
Author

p6rt commented Oct 16, 2014

From @usev6

I added a test to S03-smartmatch/capture-signature.t with the following commit​: Raku/roast@d55c943637

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