Fix some problems in the output of List, MList & TList 24/1924/2
Sylvestre Ledru [Tue, 14 Sep 2010 14:52:41 +0000 (16:52 +0200)]
Change-Id: I71403dd6c3fd572c9ccfef8a81591618f2630cf3

scilab/modules/types/src/java/org/scilab/modules/types/ScilabList.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabMList.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabTList.java

index 8835af3..8bf984d 100644 (file)
@@ -32,7 +32,7 @@ public class ScilabList extends ArrayList<ScilabType> implements ScilabType {
         * the order that they are returned by the specified collection's iterator.
         * 
         * @param c
-        *            the collection whose elements are to be placed into this list.
+        *                      the collection whose elements are to be placed into this list.
         */
        public ScilabList(Collection< ? extends ScilabType> c) {
                super(c);
@@ -42,29 +42,30 @@ public class ScilabList extends ArrayList<ScilabType> implements ScilabType {
         * @return 1 when there is data on the list, 0 otherwise.
         * @see org.scilab.modules.types.ScilabType#getHeight()
         */
-    public int getHeight() {
+       public int getHeight() {
                if (isEmpty()) {
                        return 0;
                }
                return 1;
-    }
+       }
 
-    /**
-     * @return 1 when there is data on the list, 0 otherwise.
-     * @see org.scilab.modules.types.ScilabType#getWidth()
-     */
-    public int getWidth() {
+       /**
+        * @return 1 when there is data on the list, 0 otherwise.
+        * @see org.scilab.modules.types.ScilabType#getWidth()
+        */
+       public int getWidth() {
                if (isEmpty()) {
                        return 0;
                }
                return size();
-    }
+       }
 
-    /**
-     * @return the pretty-printed data 
-     * @see java.util.AbstractCollection#toString()
-     */
-    public String toString() {
+       /**
+        * Display the representation in the Scilab language of the type
+        * @return the pretty-printed data 
+        * @see java.util.AbstractCollection#toString()
+        */
+       public String toString() {
 
                StringBuffer result = new StringBuffer();
                if (isEmpty()) {
@@ -76,11 +77,11 @@ public class ScilabList extends ArrayList<ScilabType> implements ScilabType {
                for (int i = 0; i < size(); i++) {
                        result.append(get(i).toString());
                        if (i != size() - 1) {
-                               result.append(",\n");
+                               result.append(", ");
                        }
                }
                result.append(")");
 
                return result.toString();
-    }
+       }
 }
index e163e2b..040c348 100644 (file)
@@ -26,46 +26,46 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
         * Note that the first element of this collection is the header used by
         * Scilab to find each field name.
         */
-    public ScilabMList() {
+       public ScilabMList() {
                super();
-    }
-    
-    /**
-     * Construct an mlist with a specified header.
-     *  
-     * @param types field names used by the accessors.
-     */
-    public ScilabMList(String []types) {
+       }
+       
+       /**
+        * Construct an mlist with a specified header.
+        *  
+        * @param types field names used by the accessors.
+        */
+       public ScilabMList(String []types) {
                super();
                String [][] typesData = new String[1][types.length];
                typesData[0] = types;
                add(new ScilabString(typesData));
-    }
-    
+       }
+       
        /**
         * Construct a tlist containing the elements of the specified collection, in
         * the order that they are returned by the specified collection's iterator.
         * 
         * @param names
-        *            field names, used by the accessors.
+        *                      field names, used by the accessors.
         * @param c
-        *            the collection whose elements are to be placed into this
-        *            mlist.
+        *                      the collection whose elements are to be placed into this
+        *                      mlist.
+        */
+       public ScilabMList(String[] names, Collection< ? extends ScilabType> c) {
+               super(c.size() + 1);
+               
+               String[][] namesData = new String[1][names.length];
+               namesData[0] = names;
+               add(new ScilabString(namesData));
+               
+               addAll(c);
+       }
+       
+       /**
+        * @return 1 when there is data on the list, 0 otherwise.
+        * @see org.scilab.modules.types.ScilabType#getHeight()
         */
-    public ScilabMList(String[] names, Collection< ? extends ScilabType> c) {
-       super(c.size() + 1);
-       
-       String[][] namesData = new String[1][names.length];
-       namesData[0] = names;
-       add(new ScilabString(namesData));
-       
-       addAll(c);
-    }
-    
-    /**
-     * @return 1 when there is data on the list, 0 otherwise.
-     * @see org.scilab.modules.types.ScilabType#getHeight()
-     */
        public int getHeight() {
                if (isEmpty()) {
                        return 0;
@@ -74,7 +74,7 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
        }
 
        /**
-     * @return 1 when there is data on the list, 0 otherwise.
+        * @return 1 when there is data on the list, 0 otherwise.
         * @see org.scilab.modules.types.ScilabType#getWidth()
         */
        public int getWidth() {
@@ -84,10 +84,11 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
                return size();
        }
        
-    /**
-     * @return the pretty-printed data 
-     * @see java.util.AbstractCollection#toString()
-     */
+       /**
+        * Display the representation in the Scilab language of the type
+        * @return the pretty-printed data 
+        * @see java.util.AbstractCollection#toString()
+        */
        public String toString() {
                
                StringBuffer result = new StringBuffer();
@@ -96,7 +97,7 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
                        return result.toString();
                }
 
-               result.append("mlist");
+               result.append("mlist(");
                for (int i = 0; i < size(); i++) {
                        result.append(get(i).toString());
                        if (i != size() - 1) {
index 833315d..d7ff5d1 100644 (file)
@@ -26,46 +26,46 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
         * Note that the first element of this collection is the header used by
         * Scilab to find each field type.
         */
-    public ScilabTList() {
+       public ScilabTList() {
                super();
-    }
-    
-    /**
-     * Construct a tlist with a specified header.
-     *  
-     * @param types type names of the fields.
-     */
-    public ScilabTList(String []types) {
+       }
+       
+       /**
+        * Construct a tlist with a specified header.
+        *  
+        * @param types type names of the fields.
+        */
+       public ScilabTList(String []types) {
                super();
                String [][] typesData = new String[1][types.length];
                typesData[0] = types;
                add(new ScilabString(typesData));
-    }
+       }
 
        /**
         * Construct a tlist containing the elements of the specified collection, in
         * the order that they are returned by the specified collection's iterator.
         * 
         * @param types
-        *            type names of the fields.
+        *                      type names of the fields.
         * @param c
-        *            the collection whose elements are to be placed into this
-        *            tlist.
+        *                      the collection whose elements are to be placed into this
+        *                      tlist.
+        */
+       public ScilabTList(String[] types, Collection< ? extends ScilabType> c) {
+               super(c.size() + 1);
+               
+               String[][] typesData = new String[1][types.length];
+               typesData[0] = types;
+               add(new ScilabString(typesData));
+               
+               addAll(c);
+       }
+       
+       /**
+        * @return 1 when there is data on the list, 0 otherwise.
+        * @see org.scilab.modules.types.ScilabType#getHeight()
         */
-    public ScilabTList(String[] types, Collection< ? extends ScilabType> c) {
-       super(c.size() + 1);
-       
-       String[][] typesData = new String[1][types.length];
-       typesData[0] = types;
-       add(new ScilabString(typesData));
-       
-       addAll(c);
-    }
-    
-    /**
-     * @return 1 when there is data on the list, 0 otherwise.
-     * @see org.scilab.modules.types.ScilabType#getHeight()
-     */
        public int getHeight() {
                if (isEmpty()) {
                        return 0;
@@ -74,7 +74,7 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
        }
 
        /**
-     * @return 1 when there is data on the list, 0 otherwise.
+        * @return 1 when there is data on the list, 0 otherwise.
         * @see org.scilab.modules.types.ScilabType#getWidth()
         */
        public int getWidth() {
@@ -85,6 +85,7 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
        }
        
        /**
+        * Display the representation in the Scilab language of the type
         * @return the pretty-printed data
         * @see java.util.AbstractCollection#toString()
         */