On 22 Sep 2015, at 19:10, Michael Haupt <[email protected]> wrote:
> Dear all, > > please review this change. > Issue: https://bugs.openjdk.java.net/browse/JDK-8136931 > Webrev: http://cr.openjdk.java.net/~mhaupt/8136931/webrev.00 > > This is actually a change to improve error reporting for identifying the > issue causing https://bugs.openjdk.java.net/browse/JDK-8131129, turning > assertions into actual checks. As the issue 8131129 is intermittent and > cannot be easily reproduced, the scope of additional tracing should be > minimal. Enabling all assertions, even for the BoundMethodHandle class, is > not helpful. > Does enabling assertions makes the problem harder to reproduce, and/or you need some more readable output? > This patch will be removed once the issue itself has been addressed. > +1 Suggest adding a comment to stating this is a temporary property that will be removed: 54 static final boolean OBSERVE_BMH_SPECIES_CREATION; Paul.
