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

very long right shift gratuitously fails #4880

Open
p6rt opened this issue Dec 17, 2015 · 3 comments
Open

very long right shift gratuitously fails #4880

p6rt opened this issue Dec 17, 2015 · 3 comments
Labels
LTA Less Than Awesome; typically an error message that could be better

Comments

@p6rt
Copy link

p6rt commented Dec 17, 2015

Migrated from rt.perl.org#126941 (status was 'open')

Searchable as RT126941$

@p6rt
Copy link
Author

p6rt commented Dec 17, 2015

From zefram@fysh.org

123 +> (1 +< 64)
Cannot unbox 65 bit wide bigint into native integer

This is less than awesome. The correct answer for a too-big-to-unbox
right-shift distance is trivial to obtain and to represent, so this
could easily return the correct answer.

-zefram

@p6rt
Copy link
Author

p6rt commented Dec 17, 2015

From @lizmat

On 17 Dec 2015, at 03​:37, Zefram (via RT) <perl6-bugs-followup@​perl.org> wrote​:

# New Ticket Created by Zefram
# Please include the string​: [perl #​126941]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=126941 >

123 +> (1 +< 64)
Cannot unbox 65 bit wide bigint into native integer

This is less than awesome. The correct answer for a too-big-to-unbox
right-shift distance is trivial to obtain and to represent, so this
could easily return the correct answer.

Made error a little more awesome in f6091476486d29c8886da .

$ 6 'say 123 +< (1 +< 64)'
Big integer 18446744073709551616 not yet supported on righthand side of '+<‘
operator

I assume this will want to be fixed at either the NQP level, or perhaps in rakudo core. But probably not before 6.c, so felt like a better error message is the best we can do atm.

Liz

@p6rt
Copy link
Author

p6rt commented Dec 17, 2015

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

@p6rt p6rt added the LTA Less Than Awesome; typically an error message that could be better label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LTA Less Than Awesome; typically an error message that could be better
Projects
None yet
Development

No branches or pull requests

1 participant