You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2010/02/16 17:24:34 UTC

svn commit: r910578 - /cxf/trunk/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperCompiler.java

Author: dkulp
Date: Tue Feb 16 16:24:34 2010
New Revision: 910578

URL: http://svn.apache.org/viewvc?rev=910578&view=rev
Log:
[CXF-2671] Use SIPUSH instead of BIPUSH to increase the # of arguments
supportable from 128 to 32K.   Hopefully enough.   :-)

Modified:
    cxf/trunk/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperCompiler.java

Modified: cxf/trunk/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperCompiler.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperCompiler.java?rev=910578&r1=910577&r2=910578&view=diff
==============================================================================
--- cxf/trunk/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperCompiler.java (original)
+++ cxf/trunk/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperCompiler.java Tue Feb 16 16:24:34 2010
@@ -263,7 +263,7 @@
                                       "L" + periodToSlashes(objectFactoryClass.getName()) + ";");
                 }
                 mv.visitVarInsn(Opcodes.ALOAD, 1);
-                mv.visitIntInsn(Opcodes.BIPUSH, x);
+                mv.visitIntInsn(Opcodes.SIPUSH, x);
                 mv.visitMethodInsn(Opcodes.INVOKEINTERFACE, "java/util/List", "get", "(I)Ljava/lang/Object;");
                 
                 if (tp.isPrimitive()) {
@@ -320,7 +320,7 @@
                            getMethodSignature(getMethods[x]));
         mv.visitVarInsn(Opcodes.ASTORE, 3);
         mv.visitVarInsn(Opcodes.ALOAD, 1);
-        mv.visitIntInsn(Opcodes.BIPUSH, x);
+        mv.visitIntInsn(Opcodes.SIPUSH, x);
         mv.visitMethodInsn(Opcodes.INVOKEINTERFACE, "java/util/List",
                            "get", "(I)Ljava/lang/Object;");
         mv.visitTypeInsn(Opcodes.CHECKCAST, "java/util/List");