2 Jun
2015
2 Jun
'15
8:57 a.m.
Thanks! On 02.06.2015 11:54, Remi Forax wrote:
Looks Ok to me, thumb up :)
Rémi
On 06/01/2015 08:53 PM, Ivan Gerasimov wrote:
On 01.06.2015 11:33, Paul Sandoz wrote:
On May 31, 2015, at 6:03 PM, Ivan Gerasimov <ivan.gerasimov@oracle.com> wrote:
Which is right here: http://cr.openjdk.java.net/~igerasim/8058779/05/webrev/
Much better.
For the test can you use RandomFactory recently added to the test library?
Sure. Here the updated webrev with this change and a few other minor changes. http://cr.openjdk.java.net/~igerasim/8058779/06/webrev/
The changes are: - move declaration of i below, - indent .append(), - use RandomFactory in the test, - extend number of test cases with null input.
Do you think it's ready to go?
Sincerely yours, Ivan
Paul.