Fix a syntax error and trivial refactor (cherry picked from commit 50ebdf1)
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/296bf35e Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/296bf35e Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/296bf35e Branch: refs/heads/GROOVY_2_6_X Commit: 296bf35efb5fd77f4b4ea6150d46a7f102e25733 Parents: 5966aad Author: sunlan <[email protected]> Authored: Mon Mar 5 18:22:29 2018 +0800 Committer: sunlan <[email protected]> Committed: Tue Mar 6 07:51:12 2018 +0800 ---------------------------------------------------------------------- .../org/codehaus/groovy/classgen/asm/CallSiteWriter.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/groovy/blob/296bf35e/src/main/java/org/codehaus/groovy/classgen/asm/CallSiteWriter.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/codehaus/groovy/classgen/asm/CallSiteWriter.java b/src/main/java/org/codehaus/groovy/classgen/asm/CallSiteWriter.java index 334eebd..d4ddc94 100644 --- a/src/main/java/org/codehaus/groovy/classgen/asm/CallSiteWriter.java +++ b/src/main/java/org/codehaus/groovy/classgen/asm/CallSiteWriter.java @@ -67,13 +67,13 @@ import static org.objectweb.asm.Opcodes.RETURN; * use this class in your code */ public class CallSiteWriter { - - private static String [] sig = new String [255]; + private static final int SIG_ARRAY_LENGTH = 255; + private static String [] sig = new String [SIG_ARRAY_LENGTH]; private static String getCreateArraySignature(int numberOfArguments) { - if (numberOfArguments >= 255) { + if (numberOfArguments >= SIG_ARRAY_LENGTH) { throw new IllegalArgumentException(String.format( - "The max number of arguments is 255, actual got %s", - numberOfArguments); + "The max number of arguments is %s, actual got %s", + SIG_ARRAY_LENGTH, numberOfArguments)); } if (sig[numberOfArguments] == null) { StringBuilder sb = new StringBuilder("(");
