[OpenJDK 2D-Dev] <AWT Dev> JDK-8041679 Replace uses of StringBuffer with StringBuilder within the JDK
Alan Bateman
Alan.Bateman at oracle.com
Wed May 14 09:05:41 UTC 2014
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 unless there's a compelling reason - and I don't see one.
> "buf" no longer corresponds to a buffer but to a builder, so i think it is reasonable in this case to use the canonical "sb" name.
I agree, "buf" or "buffer" looks odd when the type is changed to
StringBuilder.
> :
>
> 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:
>
> http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-media/webrev/
>
> exactly which ones should be pushed to the client forest and which ones to the jdk9-dev forest? all of them, or just a sub-set?
>
> How long would it take for the changes to the client jdk repo to make it's way into the jdk9-dev jdk repo?
>
> Really what i am getting at here is i think we need to change our process of how we manage such forests; we need to consolidate.
>
I think Sergey and Phil are asking for the bean classes from -core patch
and all of -media to be pushed to jdk9/client.
From an approachability perspective then it's total baloney for library
changes to take different routes. On the other hand there may be
exceptions needed for areas or changes that involve manual or other
special testing before integration. There was a lengthy thread on
jdk9-dev about the integration forests and processes [1] where this was
discussed. I don't recall seeing a boolean result to the questions that
asked whether the client libraries require any manual testing before
integration. As things stand then what we loosely call "client
libraries" are pushed to jdk9/client, everything else is pushed to
jdk9/dev. I really hope this can be re-visited soon.
-Alan
[1]
http://mail.openjdk.java.net/pipermail/jdk9-dev/2013-November/000000.html
More information about the core-libs-dev
mailing list