Thank you Roger and Aleksey for reviews!
On 10.08.2016 16:43, Roger Riggs wrote:
Hi Ivan,
The change looks fine.
For the tests, is there an @requires for memory that would allow the
test to run if the heap was really large.
Mostly it would have the same effect at @ignore but would be explicit
about its requirements.
Good point.
I've just found out that there's os.maxMemory property recognized by
@requires tag, so I used it in place of @ignored.
Additionally, yet another source of numeric overflow was just discovered
(that is in CharArrayReader.skip()), so here's the updated webrev.
http://cr.openjdk.java.net/~igerasim/8163518/01/webrev/
Would you please help review it once again?
With kind regards,
Ivan
