Closed GoogleCodeExporter closed 9 years ago
Just noticed a bug in skip(); the return value can be wrong. This:
// count == start at this point. Start consuming.
setConsuming(true);
n -= wasSkipped;
}
should be this:
// count == start at this point. Start consuming.
setConsuming(true);
return wasSkipped + super.skip(n - wasSkipped);
}
Original comment by tv@duh.org
on 6 Apr 2011 at 12:40
Afterthought: This could be trivially reworked to be non-abstract and use a
ByteProcessor as the data processing engine, if that's more desirable.
Original comment by tv@duh.org
on 6 Apr 2011 at 2:59
Original comment by kevinb@google.com
on 13 Jul 2011 at 7:36
Original comment by cpov...@google.com
on 13 Jul 2011 at 7:43
Original comment by fry@google.com
on 10 Dec 2011 at 4:05
Original comment by fry@google.com
on 16 Feb 2012 at 7:17
Original comment by kevinb@google.com
on 30 May 2012 at 7:43
Original comment by kevinb@google.com
on 22 Jun 2012 at 6:16
This issue has been migrated to GitHub.
It can be found at https://github.com/google/guava/issues/<id>
Original comment by cgdecker@google.com
on 1 Nov 2014 at 4:15
Original comment by cgdecker@google.com
on 1 Nov 2014 at 4:18
Original comment by cgdecker@google.com
on 3 Nov 2014 at 9:09
Original issue reported on code.google.com by
tv@duh.org
on 5 Apr 2011 at 8:29Attachments: