On 09/12/2014 15:47, Pavel Rappo wrote:
Hi everyone,

Could you please review my change for JDK-8066867?

http://cr.openjdk.java.net/~prappo/8066867/webrev.00/

In the transferTo then transferred needs to be a long. I think I would re-work the loop to make it a bit more readable but that is a somewhat subjective comment.

The test looks reasonable (from a quick skim) but the method names are inconsistent (the methods in lower case with embedded underscores).

-Alan.

Reply via email to