RFR: jsr166 openjdk9 integration
Paul Sandoz
paul.sandoz at oracle.com
Wed Sep 23 10:16:06 UTC 2015
Hi,
I trawled through the patches and could not find anything obvious that clobbered existing JDK stuff.
In the misc/locks patches there are some classes that might contain spec changes:
ConcurrentLinkedDeque
CopyOnWriteArraySet
ScheduledExecutorService
ScheduledThreadPoolExecutor
LockSupport
ReentrantReadWriteLock
Any opinions? sometimes it’s a fine line between a clarification and a spec update.
There are some new methods we need to track in the following classes:
LinkedTransferQueue
SynchronousQueue
In ThreadPoolExecutor there is style used to declare a set of fonts:
249 * <dd style="font-family:'DejaVu Sans', Arial, Helvetica, sans-serif">
250 * This class provides {@code protected} overridable
251 * {@link #beforeExecute(Thread, Runnable)} and
Is that necessary?
In Helpers:
121 private static String newStringUnsafe(char[] chars) {
122 // If porting to a JDK where sun.misc.SharedSecrets is not
123 // available, modify the code below to simply:
124 // return new String(chars);
125 // TODO: Can we do this more portably?
126 return sun.misc.SharedSecrets.getJavaLangAccess().newStringUnsafe(chars);
127 }
Yes, you can do this more portably *and* safely by not using it! :-)
Do we really really need to use SharedSecrets? IMO this unsafe dependency should be removed in the JDK patch.
Paul.
On 21 Sep 2015, at 20:34, Martin Buchholz <martinrb at google.com> wrote:
> This is the long-delayed and long-awaited bulk integration for jdk9 from jsr166 CVS.
>
> Find webrevs here:
> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/jsr166-jdk9-integration/
>
> Sorry about the extreme size and tardiness of this integration.
>
> As a review convenience, I provided a diff-wbB file that contains all the jsr166 integration changes using "hg diff -wbB" that ignores whitespace changes.
>
> We will need JPRT+specdiff+CCC from Oracle folks
>
> All changes are subtasks of:
> https://bugs.openjdk.java.net/browse/JDK-8132960
More information about the core-libs-dev
mailing list