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
Failure has no type check in constructor which leads to LTA errors #2947
Comments
From @bbkrRakudo 2012.09 $ perl6 -e 'Failure.new("foo").();' Those LTA errors can be avoided by adding Exception (or some kind of "throwable" if there is any) type check to Failure constructor. |
From @usev6Locally I simply added a type constraint to method new for class Failure: $ diff src/core/Failure.pm Failure.pm.orig
With that change the error message became: $ perl6 -e 'Failure.new("foo").();' Would that be good enough? |
1 similar comment
From @usev6Locally I simply added a type constraint to method new for class Failure: $ diff src/core/Failure.pm Failure.pm.orig
With that change the error message became: $ perl6 -e 'Failure.new("foo").();' Would that be good enough? |
@usev6 - Status changed from 'new' to 'open' |
From @usev6I added a test to S04-exceptions/fail.t with commit Raku/roast@4366d917a5 and created a pull request for Rakudo: rakudo/rakudo#442 |
1 similar comment
From @usev6I added a test to S04-exceptions/fail.t with commit Raku/roast@4366d917a5 and created a pull request for Rakudo: rakudo/rakudo#442 |
From @usev6The test passes now. I'm closing this ticket as 'resolved'. |
@usev6 - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#115436 (status was 'resolved')
Searchable as RT115436$
The text was updated successfully, but these errors were encountered: