Re: RFR: 8017231: Add StringJoiner.merge

2013-07-03 Thread Henry Jen
On Jul 3, 2013, at 9:57 AM, Henry Jen wrote: > > On Jul 3, 2013, at 9:09 AM, Alan Bateman wrote: > >> On 03/07/2013 09:03, Henry Jen wrote: >>> Hi, >>> >>> Please review a simple addition of StringJoiner.merge method. This is >>> useful to combine StringJoiners used in parallel doing joining

Re: RFR: 8017231: Add StringJoiner.merge

2013-07-03 Thread Henry Jen
On Jul 3, 2013, at 9:09 AM, Alan Bateman wrote: > On 03/07/2013 09:03, Henry Jen wrote: >> Hi, >> >> Please review a simple addition of StringJoiner.merge method. This is >> useful to combine StringJoiners used in parallel doing joining works. >> >> The webrev can be found at >> http://cr.open

Re: RFR: 8017231: Add StringJoiner.merge

2013-07-03 Thread Alan Bateman
On 03/07/2013 09:03, Henry Jen wrote: Hi, Please review a simple addition of StringJoiner.merge method. This is useful to combine StringJoiners used in parallel doing joining works. The webrev can be found at http://cr.openjdk.java.net/~henryjen/ccc/8017231.0/webrev/ Also included is a little

Re: RFR: 8017231: Add StringJoiner.merge

2013-07-03 Thread Paul Sandoz
On Jul 3, 2013, at 10:03 AM, Henry Jen wrote: > Hi, > > Please review a simple addition of StringJoiner.merge method. This is > useful to combine StringJoiners used in parallel doing joining works. > > The webrev can be found at > http://cr.openjdk.java.net/~henryjen/ccc/8017231.0/webrev/ >

RFR: 8017231: Add StringJoiner.merge

2013-07-03 Thread Henry Jen
Hi, Please review a simple addition of StringJoiner.merge method. This is useful to combine StringJoiners used in parallel doing joining works. The webrev can be found at http://cr.openjdk.java.net/~henryjen/ccc/8017231.0/webrev/ Also included is a little clean up for StringJoinerTest. Cheers,