Mercurial > hg > graal-compiler
changeset 18656:de362aafce05
fixed findbugs issue
author | Doug Simon <doug.simon@oracle.com> |
---|---|
date | Tue, 09 Dec 2014 23:53:43 +0100 |
parents | 86269e451920 |
children | 74f612049e4e |
files | graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotVMConfigVerifier.java |
diffstat | 1 files changed, 6 insertions(+), 7 deletions(-) [+] |
line wrap: on
line diff
--- 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 {