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

IO::Handle.lines doesn't properly count .ins #3549

Closed
p6rt opened this issue Oct 12, 2014 · 7 comments
Closed

IO::Handle.lines doesn't properly count .ins #3549

p6rt opened this issue Oct 12, 2014 · 7 comments

Comments

@p6rt
Copy link

p6rt commented Oct 12, 2014

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

Searchable as RT122963$

@p6rt
Copy link
Author

p6rt commented Oct 12, 2014

From @pmichaud

  pmichaud@​plum​:~/p6/rakudo$ ./perl6 --version
  This is perl6 version 2014.09-182-gd329723 built on MoarVM version 2014.09-14-g0df2d6f
  pmichaud@​plum​:~/p6/rakudo$ cat ins.p6
  my $fh = open("README.md");
 
  for $fh.lines() {
  say $fh.ins;
  last;
  }
 
 
  pmichaud@​plum​:~/p6/rakudo$ ./perl6 ins.p6
  0
  pmichaud@​plum​:~/p6/rakudo$
 
 
Result should be "1".
 
Pm
 

@p6rt
Copy link
Author

p6rt commented Oct 12, 2014

From @pmichaud

Now fixed in 7f8d176.

Needs a spectest to close ticket.

Pm

@p6rt
Copy link
Author

p6rt commented Oct 12, 2014

From @lizmat

Fixed

On 12 Oct 2014, at 18​:11, Patrick R. Michaud (via RT) <perl6-bugs-followup@​perl.org> wrote​:

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

pmichaud@​plum​:~/p6/rakudo$ ./perl6 --version
This is perl6 version 2014.09-182-gd329723 built on MoarVM version 2014.09-14-g0df2d6f
pmichaud@​plum​:~/p6/rakudo$ cat ins.p6
my $fh = open("README.md");

for $fh.lines() {
say $fh.ins;
last;
}

pmichaud@​plum​:~/p6/rakudo$ ./perl6 ins.p6
0
pmichaud@​plum​:~/p6/rakudo$

Result should be "1".

Pm

@p6rt
Copy link
Author

p6rt commented Oct 12, 2014

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

@p6rt
Copy link
Author

p6rt commented Apr 24, 2015

From @labster

Added tests in S16-filehandles/io_in_for_loops.t.

On Sun Oct 12 10​:01​:22 2014, elizabeth wrote​:

Fixed

On 12 Oct 2014, at 18​:11, Patrick R. Michaud (via RT) <perl6-bugs-
followup@​perl.org> wrote​:

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

pmichaud@​plum​:~/p6/rakudo$ ./perl6 --version
This is perl6 version 2014.09-182-gd329723 built on MoarVM version
2014.09-14-g0df2d6f
pmichaud@​plum​:~/p6/rakudo$ cat ins.p6
my $fh = open("README.md");

for $fh.lines() {
say $fh.ins;
last;
}

pmichaud@​plum​:~/p6/rakudo$ ./perl6 ins.p6
0
pmichaud@​plum​:~/p6/rakudo$

Result should be "1".

Pm

@p6rt
Copy link
Author

p6rt commented Apr 24, 2015

From @labster

Resolving ticket.

On Fri Apr 24 15​:13​:28 2015, labster wrote​:

Added tests in S16-filehandles/io_in_for_loops.t.

On Sun Oct 12 10​:01​:22 2014, elizabeth wrote​:

Fixed

On 12 Oct 2014, at 18​:11, Patrick R. Michaud (via RT) <perl6-bugs-
followup@​perl.org> wrote​:

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

pmichaud@​plum​:~/p6/rakudo$ ./perl6 --version
This is perl6 version 2014.09-182-gd329723 built on MoarVM version
2014.09-14-g0df2d6f
pmichaud@​plum​:~/p6/rakudo$ cat ins.p6
my $fh = open("README.md");

for $fh.lines() {
say $fh.ins;
last;
}

pmichaud@​plum​:~/p6/rakudo$ ./perl6 ins.p6
0
pmichaud@​plum​:~/p6/rakudo$

Result should be "1".

Pm

@p6rt
Copy link
Author

p6rt commented Apr 24, 2015

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

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