fix jni bug about Integer instead of int and scilabview does not check item validity... 71/12971/1
Antoine ELIAS [Mon, 21 Oct 2013 12:19:33 +0000 (14:19 +0200)]
Change-Id: I2fc4abdbda74618d0c385885bb77b1d9595cc510

scilab/modules/graphic_export/src/java/org/scilab/modules/graphic_export/Export.java
scilab/modules/gui/src/java/org/scilab/modules/gui/SwingView.java

index c422d44..3110452 100644 (file)
@@ -128,7 +128,7 @@ public class Export {
      * @param params the export paramaters
      * @return the export status
      */
-    public static int export(Integer uid, int type, String fileName, ExportParams params, boolean headless) {
+    public static int export(int uid, int type, String fileName, ExportParams params, boolean headless) {
         // Check that the fileName contains an extension
         int dotPosition = fileName.lastIndexOf('.'); // position of the dot
         boolean extensionFound = false;
@@ -168,7 +168,7 @@ public class Export {
      * @param params the export paramaters
      * @return the export status
      */
-    public static int exportVectorial(Integer uid, int type, String fileName, ExportParams params, boolean headless) {
+    public static int exportVectorial(int uid, int type, String fileName, ExportParams params, boolean headless) {
         if (fileName == null) {
             return INVALID_FILE;
         }
@@ -193,7 +193,7 @@ public class Export {
      * @param file the file where to export
      * @param params the export paramaters
      */
-    public static int exportVectorial(Integer uid, TYPE type, File file, ExportParams params, boolean headless) throws IOException {
+    public static int exportVectorial(int uid, TYPE type, File file, ExportParams params, boolean headless) throws IOException {
         Figure figure = (Figure) GraphicController.getController().getObjectFromId(uid);
 
         if (!headless) {
@@ -268,7 +268,7 @@ public class Export {
      * @param params the export paramaters
      * @return the export status
      */
-    public static int exportBitmap(Integer uid, int type, String fileName, boolean fromScreen, ExportParams params) {
+    public static int exportBitmap(int uid, int type, String fileName, boolean fromScreen, ExportParams params) {
         if (fileName == null) {
             return INVALID_FILE;
         }
@@ -296,7 +296,7 @@ public class Export {
      * @param fromScreen if true, then use the screen view
      * @param params the export paramaters
      */
-    public static void exportBitmap(Integer uid, TYPE type, File file, boolean fromScreen, ExportParams params) throws IOException {
+    public static void exportBitmap(int uid, TYPE type, File file, boolean fromScreen, ExportParams params) throws IOException {
         if (isBitmapFormat(type)) {
             JoGLCanvas joglCanvas = null;
             if (fromScreen) {
@@ -345,7 +345,7 @@ public class Export {
      * @param file the file where to export
      * @param params the export paramaters
      */
-    public static void setVisitor(Integer uid, int type, final ExportParams params) {
+    public static void setVisitor(int uid, int type, final ExportParams params) {
         final Exporter exporter = getExporter(types[type]);
         Figure figure = (Figure) GraphicController.getController().getObjectFromId(uid);
         final Integer[] dims = figure.getAxesSize();
index 210e79c..daa8e47 100644 (file)
@@ -626,12 +626,14 @@ public final class SwingView implements GraphicView {
     public void updateObject(final Integer id, final int property) {
         final TypedObject registeredObject = allObjects.get(id);
 
-        if (property == __GO_VALID__ && ((Boolean) GraphicController.getController().getProperty(id, __GO_VALID__))) {
-            if (registeredObject.getValue() instanceof SwingScilabTab){
+        if (registeredObject != null &&
+                property == __GO_VALID__ &&
+                ((Boolean) GraphicController.getController().getProperty(id, __GO_VALID__))) {
+            if (registeredObject.getValue() instanceof SwingScilabTab) {
                 ((SwingScilabTab) registeredObject.getValue()).getParentWindow().setVisible(true);
                 ((SwingScilabTab) registeredObject.getValue()).setVisible(true);
                 Integer[] figureSize = (Integer[]) GraphicController.getController().getProperty(id, __GO_SIZE__);
-                ((SwingScilabTab) registeredObject.getValue()).getParentWindow().setDims(new Size(figureSize[0], figureSize[1]));                
+                ((SwingScilabTab) registeredObject.getValue()).getParentWindow().setDims(new Size(figureSize[0], figureSize[1]));
             }
         }