Types: protect the ScilabList.toString() method to any null data 37/3037/2
Clément DAVID [Wed, 26 Jan 2011 13:52:10 +0000 (14:52 +0100)]
On error data can be null, protect the trace against NPEs.

Change-Id: I6852e382b6685613f2f139e3fd78be3f2d3068c2

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 a7ca365..8a7f673 100644 (file)
@@ -50,6 +50,7 @@ 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()
         */
+       @Override
        public int getHeight() {
                if (isEmpty()) {
                        return 0;
@@ -61,6 +62,7 @@ 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#getWidth()
         */
+       @Override
        public int getWidth() {
                if (isEmpty()) {
                        return 0;
@@ -75,6 +77,7 @@ public class ScilabList extends ArrayList<ScilabType> implements ScilabType {
         * @return the pretty-printed data 
         * @see java.util.AbstractCollection#toString()
         */
+       @Override
        public String toString() {
 
                StringBuffer result = new StringBuffer();
@@ -85,7 +88,7 @@ public class ScilabList extends ArrayList<ScilabType> implements ScilabType {
 
                result.append("list(");
                for (int i = 0; i < size(); i++) {
-                       result.append(get(i).toString());
+                       result.append(get(i));
                        if (i != size() - 1) {
                                result.append(", ");
                        }
index 9fffa16..516f823 100644 (file)
@@ -74,6 +74,7 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
         * @return 1 when there is data on the list, 0 otherwise.
         * @see org.scilab.modules.types.ScilabType#getHeight()
         */
+       @Override
        public int getHeight() {
                if (isEmpty()) {
                        return 0;
@@ -85,6 +86,7 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
         * @return 1 when there is data on the list, 0 otherwise.
         * @see org.scilab.modules.types.ScilabType#getWidth()
         */
+       @Override
        public int getWidth() {
                if (isEmpty()) {
                        return 0;
@@ -99,6 +101,7 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
         * @return the pretty-printed data 
         * @see java.util.AbstractCollection#toString()
         */
+       @Override
        public String toString() {
                
                StringBuffer result = new StringBuffer();
@@ -109,7 +112,7 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
 
                result.append("mlist(");
                for (int i = 0; i < size(); i++) {
-                       result.append(get(i).toString());
+                       result.append(get(i));
                        if (i != size() - 1) {
                                result.append(", ");
                        }
index a459b35..ec1a450 100644 (file)
@@ -74,6 +74,7 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
         * @return 1 when there is data on the list, 0 otherwise.
         * @see org.scilab.modules.types.ScilabType#getHeight()
         */
+       @Override
        public int getHeight() {
                if (isEmpty()) {
                        return 0;
@@ -85,6 +86,7 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
         * @return 1 when there is data on the list, 0 otherwise.
         * @see org.scilab.modules.types.ScilabType#getWidth()
         */
+       @Override
        public int getWidth() {
                if (isEmpty()) {
                        return 0;
@@ -99,6 +101,7 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
         * @return the pretty-printed data
         * @see java.util.AbstractCollection#toString()
         */
+       @Override
        public String toString() {
                
                StringBuffer result = new StringBuffer();
@@ -109,7 +112,7 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
 
                result.append("tlist(");
                for (int i = 0; i < size(); i++) {
-                       result.append(get(i).toString());
+                       result.append(get(i));
                        if (i != size() - 1) {
                                result.append(", ");
                        }