Xcos GUI: cleanup Java code 96/17496/4
Paul Bignier [Tue, 24 Nov 2015 15:28:45 +0000 (16:28 +0100)]
 * Also fixed a typo in the jni file
 * Used JavaContoller's lookup_view instead of Controller's look_for_view in ContextUpdate.java

Change-Id: I5a8ab25e884a10503c0ec41c305b8fec1f6797fa

12 files changed:
scilab/modules/xcos/src/java/org/scilab/modules/xcos/JavaController.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/actions/BlockParametersAction.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/ContextUpdate.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/graph/swing/handler/GraphHandler.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/io/codec/BasicLinkCodec.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/io/codec/XcosObjectCodec.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/io/sax/LinkHandler.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/io/writer/DelegatingXMLStreamWriter.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/io/writer/IndentingXMLStreamWriter.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/port/input/ExplicitInputPort.java
scilab/modules/xcos/src/jni/JavaController.i
scilab/modules/xcos/src/jni/JavaController_wrap.cxx

index ad38a3e..52f8ce7 100644 (file)
@@ -13,30 +13,30 @@ import java.util.TreeMap;
 
 public class JavaController extends Controller {
 
-  // will contains all registered JavaViews to prevent garbage-collection 
-  private static Map<String, View> references = new TreeMap<String, View>();
-  
-  private static long add_reference(String name, View v) {
-    references.put(name, v);
-    return View.getCPtr(v);
-  }
-
-  private static View remove_reference(View v) {
-    references.values().remove(v);
-    return v;
-  }
-
-  public static View lookup_view(String name) {
-    return references.get(name);
-  }
-
-  public static void register_view(String name, View view) {
-    JavaControllerJNI.register_view(name, add_reference(name, view), view);
-  }
-
-  public static void unregister_view(View view) {
-    JavaControllerJNI.unregister_view(View.getCPtr(view), view);
-    remove_reference(view);
-  }
+    // will contain all registered JavaViews to prevent garbage-collection
+    private static Map<String, View> references = new TreeMap<String, View>();
+
+    private static long add_reference(String name, View v) {
+        references.put(name, v);
+        return View.getCPtr(v);
+    }
+
+    private static View remove_reference(View v) {
+        references.values().remove(v);
+        return v;
+    }
+
+    public static View lookup_view(String name) {
+        return references.get(name);
+    }
+
+    public static void register_view(String name, View view) {
+        JavaControllerJNI.register_view(name, add_reference(name, view), view);
+    }
+
+    public static void unregister_view(View view) {
+        JavaControllerJNI.unregister_view(View.getCPtr(view), view);
+        remove_reference(view);
+    }
 
 }
index 651e55d..bf31649 100644 (file)
@@ -19,8 +19,6 @@ import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
 import java.awt.event.KeyEvent;
 
-import org.scilab.modules.action_binding.InterpreterManagement;
-import org.scilab.modules.action_binding.highlevel.ScilabInterpreterManagement;
 import org.scilab.modules.action_binding.highlevel.ScilabInterpreterManagement.InterpreterException;
 import org.scilab.modules.graph.ScilabGraph;
 import org.scilab.modules.graph.actions.base.VertexSelectionDependantAction;
@@ -28,7 +26,6 @@ import org.scilab.modules.gui.menuitem.MenuItem;
 import org.scilab.modules.xcos.JavaController;
 import org.scilab.modules.xcos.Kind;
 import org.scilab.modules.xcos.ObjectProperties;
-import org.scilab.modules.xcos.VectorOfScicosID;
 import org.scilab.modules.xcos.Xcos;
 import org.scilab.modules.xcos.XcosTab;
 import org.scilab.modules.xcos.block.BasicBlock;
index b3fb6e7..94a5c35 100644 (file)
@@ -292,7 +292,7 @@ public abstract class ContextUpdate extends BasicBlock {
         public static ContextUpdate createBlock(BasicPort port) {
             for (IOBlocks io : IOBlocks.values()) {
                 if (io.getReferencedPortClass().isInstance(port)) {
-                    final XcosView disabledView = (XcosView) JavaController.look_for_view(Xcos.class.getName());
+                    final XcosView disabledView = (XcosView) JavaController.lookup_view(Xcos.class.getName());
                     try {
                         JavaController controller = new JavaController();
 
index 4e384ee..a6cbbfa 100644 (file)
@@ -13,7 +13,6 @@
 package org.scilab.modules.xcos.graph.swing.handler;
 
 import java.awt.datatransfer.DataFlavor;
-import java.awt.event.ActionEvent;
 import java.awt.event.MouseEvent;
 import java.util.ArrayList;
 import java.util.List;
index 5f08dcd..9077519 100644 (file)
@@ -19,7 +19,6 @@ import java.util.Map;
 import org.scilab.modules.xcos.JavaController;
 import org.scilab.modules.xcos.Kind;
 import org.scilab.modules.xcos.block.BasicBlock;
-import org.scilab.modules.xcos.graph.model.XcosCellFactory;
 import org.scilab.modules.xcos.link.BasicLink;
 import org.scilab.modules.xcos.link.CommandControlLink;
 import org.scilab.modules.xcos.link.ExplicitLink;
index 35c10d7..2fd9f04 100644 (file)
@@ -16,8 +16,6 @@ package org.scilab.modules.xcos.io.codec;
 import java.lang.reflect.Constructor;
 import java.lang.reflect.Field;
 import java.lang.reflect.Method;
-import java.util.Collection;
-import java.util.Hashtable;
 import java.util.Map;
 import java.util.WeakHashMap;
 
@@ -26,7 +24,6 @@ import org.scilab.modules.graph.utils.StyleMap;
 import org.scilab.modules.xcos.JavaController;
 import org.scilab.modules.xcos.Kind;
 import org.scilab.modules.xcos.graph.model.XcosCell;
-import org.w3c.dom.Element;
 import org.w3c.dom.Node;
 
 import com.mxgraph.io.mxCellCodec;
index e7bdcf5..0a3741d 100644 (file)
@@ -24,7 +24,6 @@ import org.scilab.modules.xcos.link.CommandControlLink;
 import org.scilab.modules.xcos.link.ExplicitLink;
 import org.scilab.modules.xcos.link.ImplicitLink;
 import org.xml.sax.Attributes;
-import org.xml.sax.SAXParseException;
 
 class LinkHandler implements ScilabHandler {
 
index 70781a9..1c43388 100644 (file)
@@ -41,130 +41,162 @@ abstract class DelegatingXMLStreamWriter implements XMLStreamWriter {
         this.writer = writer;
     }
 
+    @Override
     public void writeStartElement(String localName) throws XMLStreamException {
         writer.writeStartElement(localName);
     }
 
+    @Override
     public void writeStartElement(String namespaceURI, String localName) throws XMLStreamException {
         writer.writeStartElement(namespaceURI, localName);
     }
 
+    @Override
     public void writeStartElement(String prefix, String localName, String namespaceURI) throws XMLStreamException {
         writer.writeStartElement(prefix, localName, namespaceURI);
     }
 
+    @Override
     public void writeEmptyElement(String namespaceURI, String localName) throws XMLStreamException {
         writer.writeEmptyElement(namespaceURI, localName);
     }
 
+    @Override
     public void writeEmptyElement(String prefix, String localName, String namespaceURI) throws XMLStreamException {
         writer.writeEmptyElement(prefix, localName, namespaceURI);
     }
 
+    @Override
     public void writeEmptyElement(String localName) throws XMLStreamException {
         writer.writeEmptyElement(localName);
     }
 
+    @Override
     public void writeEndElement() throws XMLStreamException {
         writer.writeEndElement();
     }
 
+    @Override
     public void writeEndDocument() throws XMLStreamException {
         writer.writeEndDocument();
     }
 
+    @Override
     public void close() throws XMLStreamException {
         writer.close();
     }
 
+    @Override
     public void flush() throws XMLStreamException {
         writer.flush();
     }
 
+    @Override
     public void writeAttribute(String localName, String value) throws XMLStreamException {
         writer.writeAttribute(localName, value);
     }
 
+    @Override
     public void writeAttribute(String prefix, String namespaceURI, String localName, String value) throws XMLStreamException {
         writer.writeAttribute(prefix, namespaceURI, localName, value);
     }
 
+    @Override
     public void writeAttribute(String namespaceURI, String localName, String value) throws XMLStreamException {
         writer.writeAttribute(namespaceURI, localName, value);
     }
 
+    @Override
     public void writeNamespace(String prefix, String namespaceURI) throws XMLStreamException {
         writer.writeNamespace(prefix, namespaceURI);
     }
 
+    @Override
     public void writeDefaultNamespace(String namespaceURI) throws XMLStreamException {
         writer.writeDefaultNamespace(namespaceURI);
     }
 
+    @Override
     public void writeComment(String data) throws XMLStreamException {
         writer.writeComment(data);
     }
 
+    @Override
     public void writeProcessingInstruction(String target) throws XMLStreamException {
         writer.writeProcessingInstruction(target);
     }
 
+    @Override
     public void writeProcessingInstruction(String target, String data) throws XMLStreamException {
         writer.writeProcessingInstruction(target, data);
     }
 
+    @Override
     public void writeCData(String data) throws XMLStreamException {
         writer.writeCData(data);
     }
 
+    @Override
     public void writeDTD(String dtd) throws XMLStreamException {
         writer.writeDTD(dtd);
     }
 
+    @Override
     public void writeEntityRef(String name) throws XMLStreamException {
         writer.writeEntityRef(name);
     }
 
+    @Override
     public void writeStartDocument() throws XMLStreamException {
         writer.writeStartDocument();
     }
 
+    @Override
     public void writeStartDocument(String version) throws XMLStreamException {
         writer.writeStartDocument(version);
     }
 
+    @Override
     public void writeStartDocument(String encoding, String version) throws XMLStreamException {
         writer.writeStartDocument(encoding, version);
     }
 
+    @Override
     public void writeCharacters(String text) throws XMLStreamException {
         writer.writeCharacters(text);
     }
 
+    @Override
     public void writeCharacters(char[] text, int start, int len) throws XMLStreamException {
         writer.writeCharacters(text, start, len);
     }
 
+    @Override
     public String getPrefix(String uri) throws XMLStreamException {
         return writer.getPrefix(uri);
     }
 
+    @Override
     public void setPrefix(String prefix, String uri) throws XMLStreamException {
         writer.setPrefix(prefix, uri);
     }
 
+    @Override
     public void setDefaultNamespace(String uri) throws XMLStreamException {
         writer.setDefaultNamespace(uri);
     }
 
+    @Override
     public void setNamespaceContext(NamespaceContext context) throws XMLStreamException {
         writer.setNamespaceContext(context);
     }
 
+    @Override
     public NamespaceContext getNamespaceContext() {
         return writer.getNamespaceContext();
     }
 
+    @Override
     public Object getProperty(String name) throws IllegalArgumentException {
         return writer.getProperty(name);
     }
index 37daa89..9dfdae1 100644 (file)
@@ -61,6 +61,7 @@ public class IndentingXMLStreamWriter extends DelegatingXMLStreamWriter {
      * @deprecated
      *      Only return the length of the indent string.
      */
+    @Deprecated
     public int getIndentStep() {
         return indentStep.length();
     }
@@ -76,6 +77,7 @@ public class IndentingXMLStreamWriter extends DelegatingXMLStreamWriter {
      * @deprecated
      *      Should use the version that takes string.
      */
+    @Deprecated
     public void setIndentStep(int indentStep) {
         StringBuilder s = new StringBuilder();
         for (; indentStep > 0; indentStep--) {
@@ -131,66 +133,79 @@ public class IndentingXMLStreamWriter extends DelegatingXMLStreamWriter {
     }
 
 
+    @Override
     public void writeStartDocument() throws XMLStreamException {
         super.writeStartDocument();
         super.writeCharacters("\n");
     }
 
+    @Override
     public void writeStartDocument(String version) throws XMLStreamException {
         super.writeStartDocument(version);
         super.writeCharacters("\n");
     }
 
+    @Override
     public void writeStartDocument(String encoding, String version) throws XMLStreamException {
         super.writeStartDocument(encoding, version);
         super.writeCharacters("\n");
     }
 
+    @Override
     public void writeStartElement(String localName) throws XMLStreamException {
         onStartElement();
         super.writeStartElement(localName);
     }
 
+    @Override
     public void writeStartElement(String namespaceURI, String localName) throws XMLStreamException {
         onStartElement();
         super.writeStartElement(namespaceURI, localName);
     }
 
+    @Override
     public void writeStartElement(String prefix, String localName, String namespaceURI) throws XMLStreamException {
         onStartElement();
         super.writeStartElement(prefix, localName, namespaceURI);
     }
 
+    @Override
     public void writeEmptyElement(String namespaceURI, String localName) throws XMLStreamException {
         onEmptyElement();
         super.writeEmptyElement(namespaceURI, localName);
     }
 
+    @Override
     public void writeEmptyElement(String prefix, String localName, String namespaceURI) throws XMLStreamException {
         onEmptyElement();
         super.writeEmptyElement(prefix, localName, namespaceURI);
     }
 
+    @Override
     public void writeEmptyElement(String localName) throws XMLStreamException {
         onEmptyElement();
         super.writeEmptyElement(localName);
     }
 
+    @Override
     public void writeEndElement() throws XMLStreamException {
         onEndElement();
         super.writeEndElement();
     }
 
+    @Override
     public void writeCharacters(String text) throws XMLStreamException {
         state = SEEN_DATA;
         super.writeCharacters(text);
     }
 
+    @Override
     public void writeCharacters(char[] text, int start, int len) throws XMLStreamException {
         state = SEEN_DATA;
         super.writeCharacters(text, start, len);
     }
 
+    @Override
     public void writeCData(String data) throws XMLStreamException {
         state = SEEN_DATA;
         super.writeCData(data);
index fc4a624..811fbfe 100644 (file)
@@ -12,8 +12,6 @@
 
 package org.scilab.modules.xcos.port.input;
 
-import org.scilab.modules.xcos.PortKind;
-
 /**
  * An explicit output port is a port where the transfer of information has a
  * 'IN' direction.
index 601426f..83548e3 100644 (file)
@@ -37,7 +37,7 @@
  * Inline std_vector.i and add insert and remove methods
  */
 namespace std {
-    
+
     template<class T> class vector {
       public:
         typedef size_t size_type;
@@ -203,7 +203,7 @@ namespace std {
     SWIG_JavaThrowException(jenv, SWIG_JavaIndexOutOfBoundsException, "Array must contain at least 1 element");
     return $null;
   }
-  $1 = &temp; 
+  $1 = &temp;
   *$1 = "";
 }
 
@@ -212,7 +212,7 @@ namespace std {
   if ($1) {
      jnewstring = JCALL1(NewStringUTF, jenv, $1->c_str());
   }
-  JCALL3(SetObjectArrayElement, jenv, $input, 0, jnewstring); 
+  JCALL3(SetObjectArrayElement, jenv, $input, 0, jnewstring);
 }
 
 %apply double &OUTPUT { double &v };
@@ -310,9 +310,9 @@ import java.util.TreeMap;
 %pragma(java) modulebase="Controller"
 
 %pragma(java) modulecode=%{
-  // will contains all registered JavaViews to prevent garbage-collection 
+  // will contain all registered JavaViews to prevent garbage-collection
   private static Map<String, View> references = new TreeMap<String, View>();
-  
+
   private static long add_reference(String name, View v) {
     references.put(name, v);
     return View.getCPtr(v);
index 4e1f566..292c57d 100644 (file)
@@ -1588,7 +1588,7 @@ SWIGEXPORT jboolean JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_Contr
       SWIG_JavaThrowException(jenv, SWIG_JavaIndexOutOfBoundsException, "Array must contain at least 1 element");
       return 0;
     }
-    arg5 = &temp5; 
+    arg5 = &temp5;
     *arg5 = "";
   }
   result = (bool)((org_scilab_modules_scicos::Controller const *)arg1)->SWIGTEMPLATEDISAMBIGUATOR getObjectProperty< std::string >(arg2,arg3,arg4,*arg5);
@@ -1598,7 +1598,7 @@ SWIGEXPORT jboolean JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_Contr
     if (arg5) {
       jnewstring = jenv->NewStringUTF(arg5->c_str());
     }
-    jenv->SetObjectArrayElement(jarg5, 0, jnewstring); 
+    jenv->SetObjectArrayElement(jarg5, 0, jnewstring);
   }
   return jresult;
 }