Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-06-16 Thread Paul Sandoz
After some further reflection i have decided to not separate this out and some bits going to client and some bits going to dev, and have pushed all changes to dev. I realise this might cause some friction but believe it the right thing to do. I remain unconvinced that the separate process for

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-20 Thread Paul Sandoz
On May 19, 2014, at 6:18 PM, Phil Race philip.r...@oracle.com wrote: On 5/19/2014 12:50 AM, Alan Bateman wrote: On 19/05/2014 07:53, Paul Sandoz wrote: If i don't have permission to push to the client repo (which might be likely) i will need to hand over the patch to yourself or Sergey to

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-19 Thread Paul Sandoz
On May 16, 2014, at 8:33 PM, Phil Race philip.r...@oracle.com wrote: On 5/16/2014 5:13 AM, Paul Sandoz wrote: On May 14, 2014, at 11:05 AM, Alan Bateman alan.bate...@oracle.com wrote: : This code is under src/share/classes, so it's not clear to me what classes i can modify and push in the

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-19 Thread Alan Bateman
On 19/05/2014 07:53, Paul Sandoz wrote: If i don't have permission to push to the client repo (which might be likely) i will need to hand over the patch to yourself or Sergey to commit. And i presume this will have to be a separate issue. If you do decide to split this then it will require

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-19 Thread Phil Race
On 5/19/2014 12:50 AM, Alan Bateman wrote: On 19/05/2014 07:53, Paul Sandoz wrote: If i don't have permission to push to the client repo (which might be likely) i will need to hand over the patch to yourself or Sergey to commit. And i presume this will have to be a separate issue. If you do

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-16 Thread Paul Sandoz
On May 14, 2014, at 11:05 AM, Alan Bateman alan.bate...@oracle.com wrote: : This code is under src/share/classes, so it's not clear to me what classes i can modify and push in the jdk9-dev/jdk repo or not. It's confusing! Is there a list online somewhere? Out of all the classes here:

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-16 Thread Phil Race
On 5/16/2014 5:13 AM, Paul Sandoz wrote: On May 14, 2014, at 11:05 AM, Alan Bateman alan.bate...@oracle.com wrote: : This code is under src/share/classes, so it's not clear to me what classes i can modify and push in the jdk9-dev/jdk repo or not. It's confusing! Is there a list online

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-14 Thread Paul Sandoz
HI Phil, Thanks for looking at this. On May 13, 2014, at 11:15 PM, Phil Race philip.r...@oracle.com wrote: Paul, I don't see why you changed the variable names in some cases. Note it's not me :-) I am, mostly, the proxy. See here where one change is only one line since you left it alone

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-14 Thread Alan Bateman
On 14/05/2014 09:15, Paul Sandoz wrote: : And here what could have been a 2 line change is 25 .. http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-media/webrev/src/share/classes/sun/font/StandardTextSource.java.sdiff.html So I would say leave the variable names alone

Re: [OpenJDK 2D-Dev] AWT Dev JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK

2014-05-13 Thread Phil Race
Paul, I don't see why you changed the variable names in some cases. See here where one change is only one line since you left it alone and the other is 6 lines since you changed it