* Updated JGraphX to the "2.1.0.7" version, updating our performance hooks 16/12216/3
Clément DAVID [Fri, 2 Aug 2013 10:22:11 +0000 (12:22 +0200)]
ScilabObjectCodec caching methods has been integrated upstream however the
XcosObjectCodec shortcuts always has a huge performance impact.

Change-Id: I8bc28c7e686895c35e0069bab65db980e5cd709c

scilab/CHANGES_5.5.X
scilab/configure
scilab/configure.ac
scilab/modules/graph/src/java/org/scilab/modules/graph/io/ScilabObjectCodec.java

index f865693..c75d057 100644 (file)
@@ -98,7 +98,9 @@ SciNotes
 Xcos
 ====
 
-* Updated Sundials to lastest "2.5.0", keeping our modifications
+* Updated Sundials to the "2.5.0" version, keeping our modifications
+
+* Updated JGraphX to the "2.1.0.7" version, updating our performance hooks
 
 * Added a new DAE solver : DDaskr, using BDF methods with direct Newton and preconditioned Krylov linear solvers, which includes rootfinding.
 
index 8876a92..a80869b 100755 (executable)
@@ -12497,8 +12497,8 @@ $as_echo "$as_me: WARNING: Could not find or use the Java package/jar jgraphx us
 
 
 
-    { $as_echo "$as_me:${as_lineno-$LINENO}: checking minimal version (1.8.0.0) of jgraphx" >&5
-$as_echo_n "checking minimal version (1.8.0.0) of jgraphx... " >&6; }
+    { $as_echo "$as_me:${as_lineno-$LINENO}: checking minimal version (2.1.0.7) of jgraphx" >&5
+$as_echo_n "checking minimal version (2.1.0.7) of jgraphx... " >&6; }
    saved_ac_java_classpath=$ac_java_classpath
    export ac_java_classpath="$JGRAPHX:$ac_java_classpath"
    if test "x" == "x"; then
@@ -12511,7 +12511,7 @@ import com.mxgraph.view.mxGraph;
 
 public class conftest {
     public static void main(String[] argv) {
-        String minVersion="1.8.0.0";
+        String minVersion="2.1.0.7";
 
                             System.out.println(mxGraph.VERSION);
                                   if (compare(minVersion, mxGraph.VERSION) > 0) {
@@ -12560,7 +12560,7 @@ EOF
                    STDOUT=`cat conftest.java.output`
                 fi
                 echo "configure: CLASSPATH was $CLASSPATH" >&5
-                  as_fn_error $? "Wrong version of jgraphx. Expected at least 1.8.0.0. Found $STDOUT" "$LINENO" 5
+                  as_fn_error $? "Wrong version of jgraphx. Expected at least 2.1.0.7. Found $STDOUT" "$LINENO" 5
                             fi
         fi
         if test -f conftest.java.output; then
@@ -12570,7 +12570,7 @@ EOF
         echo "configure: failed program was:" >&5
         cat conftest.java >&5
         echo "configure: CLASSPATH was $CLASSPATH" >&5
-          as_fn_error $? "Wrong version of jgraphx. Expected at least 1.8.0.0. Found $STDOUT" "$LINENO" 5
+          as_fn_error $? "Wrong version of jgraphx. Expected at least 2.1.0.7. Found $STDOUT" "$LINENO" 5
             fi
 
    else
@@ -12583,7 +12583,7 @@ import com.mxgraph.view.mxGraph;
 
 public class conftest {
     public static void main(String[] argv) {
-        String minVersion="1.8.0.0";
+        String minVersion="2.1.0.7";
 
                             System.out.println(mxGraph.VERSION);
                                   if (compare(minVersion, mxGraph.VERSION) != 0) {
@@ -12632,7 +12632,7 @@ EOF
                    STDOUT=`cat conftest.java.output`
                 fi
                 echo "configure: CLASSPATH was $CLASSPATH" >&5
-                  as_fn_error $? "Wrong version of jgraphx. Expected exact version 1.8.0.0. Found $STDOUT" "$LINENO" 5
+                  as_fn_error $? "Wrong version of jgraphx. Expected exact version 2.1.0.7. Found $STDOUT" "$LINENO" 5
                             fi
         fi
         if test -f conftest.java.output; then
@@ -12642,7 +12642,7 @@ EOF
         echo "configure: failed program was:" >&5
         cat conftest.java >&5
         echo "configure: CLASSPATH was $CLASSPATH" >&5
-          as_fn_error $? "Wrong version of jgraphx. Expected exact version 1.8.0.0. Found $STDOUT" "$LINENO" 5
+          as_fn_error $? "Wrong version of jgraphx. Expected exact version 2.1.0.7. Found $STDOUT" "$LINENO" 5
             fi
 
     fi
index fc702cc..d332c6c 100644 (file)
@@ -860,7 +860,7 @@ if test "$with_javasci" != no -o "$with_gui" != no -o "$enable_build_help" != no
                    JGRAPHX=$PACKAGE_JAR_FILE
                    AC_SUBST(JGRAPHX)
 
-                   AC_JAVA_CHECK_VERSION_PACKAGE([jgraphx],[import com.mxgraph.view.mxGraph;],$JGRAPHX,[1.8.0.0],[mxGraph.VERSION],[],[])
+                   AC_JAVA_CHECK_VERSION_PACKAGE([jgraphx],[import com.mxgraph.view.mxGraph;],$JGRAPHX,[2.0.0.1],[mxGraph.VERSION],[],[])
 
                 fi
 
index c0dbe6d..c1fb044 100644 (file)
@@ -141,32 +141,6 @@ public abstract class ScilabObjectCodec extends mxObjectCodec {
     }
 
     /**
-     * Shortcut {@link mxObjectCodec#getMethod} for performance
-     */
-    @Override
-    protected Method getMethod(Object obj, String methodname, Class[] params) {
-        Class<?> type = obj.getClass();
-        Method method = null;
-
-        try {
-            method = type.getMethod(methodname, params);
-        } catch (SecurityException e) {
-        } catch (NoSuchMethodException e) {
-        }
-
-        return method;
-    }
-
-    /**
-     * Shortcut {@link mxObjectCodec#getField} for performance
-     */
-    @Override
-    protected Field getField(Object obj, String fieldname) {
-        // all data is accessed through the methods, not directly
-        return null;
-    }
-
-    /**
      * get an integer value from a attributes.
      *
      * @param dataAttrs