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

.rotor no longer accepts Infs (@a.rotor: 3, Inf, :partial) #5934

Closed
p6rt opened this issue Dec 29, 2016 · 3 comments
Closed

.rotor no longer accepts Infs (@a.rotor: 3, Inf, :partial) #5934

p6rt opened this issue Dec 29, 2016 · 3 comments
Labels
regression Issue did not exist previously testneeded

Comments

@p6rt
Copy link

p6rt commented Dec 29, 2016

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

Searchable as RT130436$

@p6rt
Copy link
Author

p6rt commented Dec 29, 2016

From @AlexDaniel

Code​:
my @​a = ‘a’..‘e’; say @​a.rotor(3, Inf, :partial)

Result (2015.12,2016.07)​:
((a b c) (d e))

Result (HEAD)​:
Cannot coerce Inf to an Int
  in block <unit> at /tmp/TtHxYDq4L6 line 1

Actually thrown at​:
  in block <unit> at /tmp/TtHxYDq4L6 line 1
«exit code = 1»

Bisectable points to rakudo/rakudo@59f040d

Please note that other forms were not really working. For example, change @​a.rotor to (‘a’..‘e’).rotor and Inf is no longer accepted. Still, I think Inf could be useful sometimes, we should consider making it work.

@p6rt
Copy link
Author

p6rt commented Jan 18, 2017

@lizmat - Status changed from 'new' to 'resolved'

@p6rt p6rt closed this as completed Jan 18, 2017
@p6rt
Copy link
Author

p6rt commented Jan 18, 2017

From @zoffixznet

Fixed in rakudo/rakudo@7ddc5f7

@p6rt p6rt added regression Issue did not exist previously testneeded labels Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Issue did not exist previously testneeded
Projects
None yet
Development

No branches or pull requests

1 participant