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

.lines($limit) on ArgFiles no longer uses $limit as a limit ($*ARGFILES.lines(100)) #5927

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

Comments

@p6rt
Copy link

p6rt commented Dec 29, 2016

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

Searchable as RT130430$

@p6rt
Copy link
Author

p6rt commented Dec 29, 2016

From @AlexDaniel

Instead, it returns exactly $limit results.

Code​:
say +$*ARGFILES.lines(100) # to replicate, run it with one line on stdin

Result (2015.12)​:
1

Result (HEAD)​:
100

Bisectable points to rakudo/rakudo@24f0eeb

Perhaps a better visualization of what is happening​: https://gist.github.com/Whateverable/d5a5186e85c168b1990d87bb726403c9

@p6rt
Copy link
Author

p6rt commented Dec 29, 2016

From @zoffixznet

On Wed, 28 Dec 2016 17​:38​:33 -0800, alex.jakimenko@​gmail.com wrote​:

Instead, it returns exactly $limit results.

Code​:
say +$*ARGFILES.lines(100) # to replicate, run it with one line on
stdin

Result (2015.12)​:
1

Result (HEAD)​:
100

Bisectable points to
rakudo/rakudo@24f0eeb

Perhaps a better visualization of what is happening​:
https://gist.github.com/Whateverable/d5a5186e85c168b1990d87bb726403c9

Thank you for the report. This is now fixed, along with 4 more bugs found during the fix.

Fixes​: rakudo/rakudo@19df3584e68269f719
Tests​: Raku/roast@0f53a2d412

@p6rt
Copy link
Author

p6rt commented Dec 29, 2016

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

@p6rt
Copy link
Author

p6rt commented Dec 29, 2016

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

@p6rt p6rt closed this as completed Dec 29, 2016
@p6rt p6rt added the regression Issue did not exist previously label 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
Projects
None yet
Development

No branches or pull requests

1 participant