Code Review for hs25.60-b10-jdk8u60-b10

Prepared by:amurillo on Fri Apr 3 10:25:59 PDT 2015
Workspace:hs25.60-b10-jdk8u60-b10
Compare against: jdk8u/jdk8u
Summary of changes: 52 lines changed: 25 ins; 0 del; 27 mod; 14003 unchg
Patch of changes: hs25.60-b10-jdk8u60-b10.patch
Author comments:
8055231: ZERO variant build is broken
8068007: [Findbugs] SA com.sun.java.swing.action.ActionManager.manager should be package protect
8068036: assert(is_available(index)) failed in G1 cset
8076191: new hotspot build - hs25.60-b10
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/.hgtags

rev 7202 : Added tag hs25.60-b10 for changeset 57a14c3927eb
1 line changed: 1 ins; 0 del; 0 mod; 610 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/agent/src/share/classes/com/sun/java/swing/action/ActionManager.java

rev 7200 : 8068007: [Findbugs] SA com.sun.java.swing.action.ActionManager.manager should be package protect
Summary: fixed java programming style nit
Reviewed-by: dholmes, jbachorik, sspitsyn
6 lines changed: 5 ins; 0 del; 1 mod; 94 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/agent/src/share/classes/sun/jvm/hotspot/ui/action/HSDBActionManager.java

rev 7200 : 8068007: [Findbugs] SA com.sun.java.swing.action.ActionManager.manager should be package protect
Summary: fixed java programming style nit
Reviewed-by: dholmes, jbachorik, sspitsyn
5 lines changed: 2 ins; 0 del; 3 mod; 53 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/make/hotspot_version

rev 7196 : 8076191: new hotspot build - hs25.60-b10
Reviewed-by: dholmes
1 line changed: 0 ins; 0 del; 1 mod; 44 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/zero/vm/cppInterpreter_zero.cpp

rev 7199 : 8055231: ZERO variant build is broken
Summary: Fix zero build.
Reviewed-by: coleenp
Contributed-by: Severin Gehwolf <sgehwolf@redhat.com>
1 line changed: 0 ins; 0 del; 1 mod; 1063 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/zero/vm/frame_zero.inline.hpp

rev 7199 : 8055231: ZERO variant build is broken
Summary: Fix zero build.
Reviewed-by: coleenp
Contributed-by: Severin Gehwolf <sgehwolf@redhat.com>
2 lines changed: 2 ins; 0 del; 0 mod; 175 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp

rev 7197 : 8068036: assert(is_available(index)) failed in G1 cset
Summary: Some verification code iterated over the heap using the region mapping array. This is not allowed. Changed to use the regular iteration method with closure.
Reviewed-by: jwilhelm, brutisso
32 lines changed: 12 ins; 0 del; 20 mod; 6873 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp

rev 7197 : 8068036: assert(is_available(index)) failed in G1 cset
Summary: Some verification code iterated over the heap using the region mapping array. This is not allowed. Changed to use the regular iteration method with closure.
Reviewed-by: jwilhelm, brutisso
3 lines changed: 3 ins; 0 del; 0 mod; 1633 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/interpreter/bytecodeInterpreter.cpp

rev 7199 : 8055231: ZERO variant build is broken
Summary: Fix zero build.
Reviewed-by: coleenp
Contributed-by: Severin Gehwolf <sgehwolf@redhat.com>
1 line changed: 0 ins; 0 del; 1 mod; 3458 unchg

This code review page was prepared using /export/pub/bin/webrev (vers 25.6-hg+openjdk.java.net).