Re: code review request for 6880112, Coin: use diamond in core libraries

2010-12-17 Thread Paul Benedict
Do it as two commits (one for diamond, the other for line formatting) that way you can keep the work separate. You may even want 2 Sun tracking issue in case you need to do several commits under one issue. Paul On Thu, Dec 16, 2010 at 10:43 PM, Joe Darcy joe.da...@oracle.com wrote: On

Re: code review request for 6880112, Coin: use diamond in core libraries

2010-12-17 Thread Stuart Marks
On 12/17/10 6:54 AM, Paul Benedict wrote: Do it as two commits (one for diamond, the other for line formatting) that way you can keep the work separate. You may even want 2 Sun tracking issue in case you need to do several commits under one issue. The jcheck rules *require* a separate bug ID

Re: code review request for 6880112, Coin: use diamond in core libraries

2010-12-17 Thread Joe Darcy
On 12/17/2010 11:57 AM, Stuart Marks wrote: On 12/17/10 6:54 AM, Paul Benedict wrote: Do it as two commits (one for diamond, the other for line formatting) that way you can keep the work separate. You may even want 2 Sun tracking issue in case you need to do several commits under one issue.

Re: code review request for 6880112, Coin: use diamond in core libraries

2010-12-17 Thread Mandy Chung
On 12/16/10 7:14 PM, Stuart Marks wrote: I've run the line joiner and it changed ten files. Here's an incremental webrev, over and above yesterday's: http://cr.openjdk.java.net/~smarks/reviews/6880112/webrev.1/ Looks good to me. Mandy

Re: code review request for 6880112, Coin: use diamond in core libraries

2010-12-17 Thread Mike Duigou
All changes look good to me. On Dec 16 2010, at 19:14 , Stuart Marks wrote: I've run the line joiner and it changed ten files. Here's an incremental webrev, over and above yesterday's: http://cr.openjdk.java.net/~smarks/reviews/6880112/webrev.1/

Review for CR 6728865 : Improved heuristics for Collections.disjoint()

2010-12-17 Thread Mike Duigou
I've posted a webrev for review at http://cr.openjdk.java.net/~mduigou/6728865.0/webrev/ which improves the behaviour of Collections.disjoint() when the collection c1 is not a Set and is larger than c2. http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6728865 I've included some other