--- jsr166/src/jsr166e/ConcurrentHashMapV8.java	2012/12/15 20:21:25	1.84
+++ jsr166/src/jsr166e/ConcurrentHashMapV8.java	2013/01/18 04:23:27	1.88
@@ -567,7 +567,7 @@ public class ConcurrentHashMapV8<K, V>
     static final int SEED_INCREMENT = 0x61c88647;
 
     /**
-     * Per-thread counter hash codes. Shared across all instances
+     * Per-thread counter hash codes. Shared across all instances.
      */
     static final ThreadLocal<CounterHashCode> threadCounterHashCode =
         new ThreadLocal<CounterHashCode>();
@@ -3730,7 +3730,6 @@ public class ConcurrentHashMapV8<K, V>
      * result on success, else null
      * @return a non-null result from applying the given search
      * function on each value, or null if none
-     *
      */
     public <U> U searchValuesSequentially
         (Fun<? super V, ? extends U> searchFunction) {
@@ -4337,7 +4336,6 @@ public class ConcurrentHashMapV8<K, V>
      * result on success, else null
      * @return a non-null result from applying the given search
      * function on each value, or null if none
-     *
      */
     public <U> U searchValuesInParallel
         (Fun<? super V, ? extends U> searchFunction) {
@@ -4579,7 +4577,7 @@ public class ConcurrentHashMapV8<K, V>
     /**
      * Base class for views.
      */
-    static abstract class CHMView<K, V> {
+    abstract static class CHMView<K, V> {
         final ConcurrentHashMapV8<K, V> map;
         CHMView(ConcurrentHashMapV8<K, V> map)  { this.map = map; }
 
@@ -4595,9 +4593,9 @@ public class ConcurrentHashMapV8<K, V>
         public final void clear()               { map.clear(); }
 
         // implementations below rely on concrete classes supplying these
-        abstract public Iterator<?> iterator();
-        abstract public boolean contains(Object o);
-        abstract public boolean remove(Object o);
+        public abstract Iterator<?> iterator();
+        public abstract boolean contains(Object o);
+        public abstract boolean remove(Object o);
 
         private static final String oomeMsg = "Required array size too large";
 
@@ -6844,21 +6842,23 @@ public class ConcurrentHashMapV8<K, V>
     private static sun.misc.Unsafe getUnsafe() {
         try {
             return sun.misc.Unsafe.getUnsafe();
-        } catch (SecurityException se) {
-            try {
-                return java.security.AccessController.doPrivileged
-                    (new java.security
-                     .PrivilegedExceptionAction<sun.misc.Unsafe>() {
-                        public sun.misc.Unsafe run() throws Exception {
-                            java.lang.reflect.Field f = sun.misc
-                                .Unsafe.class.getDeclaredField("theUnsafe");
-                            f.setAccessible(true);
-                            return (sun.misc.Unsafe) f.get(null);
-                        }});
-            } catch (java.security.PrivilegedActionException e) {
-                throw new RuntimeException("Could not initialize intrinsics",
-                                           e.getCause());
-            }
+        } catch (SecurityException tryReflectionInstead) {}
+        try {
+            return java.security.AccessController.doPrivileged
+            (new java.security.PrivilegedExceptionAction<sun.misc.Unsafe>() {
+                public sun.misc.Unsafe run() throws Exception {
+                    Class<sun.misc.Unsafe> k = sun.misc.Unsafe.class;
+                    for (java.lang.reflect.Field f : k.getDeclaredFields()) {
+                        f.setAccessible(true);
+                        Object x = f.get(null);
+                        if (k.isInstance(x))
+                            return k.cast(x);
+                    }
+                    throw new NoSuchFieldError("the Unsafe");
+                }});
+        } catch (java.security.PrivilegedActionException e) {
+            throw new RuntimeException("Could not initialize intrinsics",
+                                       e.getCause());
         }
     }
 }