# HG changeset patch # User Doug Simon # Date 1418165623 -3600 # Node ID de362aafce055ec0da0da9abe3d920eca008ef65 # Parent 86269e4519209615e15857ab1692daae4953026b fixed findbugs issue diff -r 86269e451920 -r de362aafce05 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotVMConfigVerifier.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotVMConfigVerifier.java Tue Dec 09 23:44:07 2014 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotVMConfigVerifier.java Tue Dec 09 23:53:43 2014 +0100 @@ -84,16 +84,15 @@ this.sourceFile = source; } - void verify(boolean condition, String name, String signature, String message) { + void verify(boolean condition, String message) { if (!condition) { - error(message, name, signature); + error(message); } } - void error(String message, String name, String signature) { - String errorMessage = format( - "%s:%d: Use of Unsafe in %s.%s%s is not allowed in the context of compilation replay. The unsafe access should be moved into the %s constructor and the result cached in a field that will be returned by %s%s%n", - sourceFile, lineNo, HotSpotVMConfig.class.getSimpleName(), name, signature, message, HotSpotVMConfig.class.getSimpleName(), name, signature); + void error(String message) { + String errorMessage = format("%s:%d: %s is not allowed in the context of compilation replay. The unsafe access should be moved into the %s constructor and the result cached in a field", + sourceFile, lineNo, message, HotSpotVMConfig.class.getSimpleName()); throw new InternalError(errorMessage); } @@ -147,7 +146,7 @@ @Override public void visitMethodInsn(int opcode, String owner, String methodName, String methodDesc, boolean itf) { Executable callee = resolveMethod(owner, methodName, methodDesc); - verify(checkInvokeTarget(callee), name, d, "invocation of " + callee); + verify(checkInvokeTarget(callee), "invocation of " + callee); } }; } else {