Coverity fixes: #1351121 #1351122 #1351123 #1351124 #1351125 #1351126 #1351127 52/19252/3
Nikhil Goel [Tue, 11 Apr 2017 13:03:38 +0000 (18:03 +0530)]
Change-Id: Ia0809f0b35ed6f36101050ac735d9a7232998330

scilab/modules/external_objects_java/src/java/org/scilab/modules/external_objects_java/ScilabJarCreator.java
scilab/modules/helptools/src/java/org/scilab/modules/helptools/FopConverter.java
scilab/modules/helptools/src/java/org/scilab/modules/helptools/SciDocMain.java
scilab/modules/helptools/src/java/org/scilab/modules/helptools/image/XcosImageConverter.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/graph/XcosDiagram.java

index 1ff9ac5..546a637 100644 (file)
@@ -240,7 +240,7 @@ public class ScilabJarCreator {
                 jarOutputStream.close();
             }
         } catch (IOException e) {
-            throw new ScilabJavaException(String.format("Cannot close jar stream: %s\n", e.getMessage()));
+            throw new ScilabJavaException(String.format("Cannot close jar stream: %s%n", e.getMessage()));
         }
     }
 
@@ -253,13 +253,13 @@ public class ScilabJarCreator {
         if (f.exists()) {
             if (f.canWrite()) {
                 if (!f.delete()) {
-                    throw new ScilabJavaException(String.format("Cannot delete jar archive %s.\n", jarFilePath));
+                    throw new ScilabJavaException(String.format("Cannot delete jar archive %s.%n", jarFilePath));
                 }
             } else {
-                throw new ScilabJavaException(String.format("Cannot delete jar archive %s: File is write protected.\n", jarFilePath));
+                throw new ScilabJavaException(String.format("Cannot delete jar archive %s: File is write protected.%n", jarFilePath));
             }
         } else {
-            throw new ScilabJavaException(String.format("Cannot delete jar archive %s: No such file.\n", jarFilePath));
+            throw new ScilabJavaException(String.format("Cannot delete jar archive %s: No such file.%n", jarFilePath));
         }
     }
 }
index ad0ebb9..a0f48ce 100644 (file)
@@ -61,7 +61,7 @@ public class FopConverter extends ContainerConverter {
 
                 default:
                     out.close();
-                    throw new IOException(String.format("%s is not a supported format.\n", format));
+                    throw new IOException(String.format("%s is not a supported format.%n", format));
             }
 
             // Step 4: Setup JAXP using identity transformer
index b015878..b982086 100644 (file)
@@ -110,7 +110,7 @@ public final class SciDocMain {
             str.append(values[i].toString().toLowerCase());
         }
         str.append(']');
-        System.err.printf("%s is not a supported format : one of %s expected.\n", format.toString(), str);
+        System.err.printf("%s is not a supported format : one of %s expected.%n", format.toString(), str);
 
     }
 
@@ -161,7 +161,7 @@ public final class SciDocMain {
                     converter = new FopConverter(this);
                     break;
                 default:
-                    System.err.printf("%s is not a supported format.\n", format);
+                    System.err.printf("%s is not a supported format.%n", format);
                     return null;
             }
 
index 54d7458..abca01a 100644 (file)
@@ -65,7 +65,7 @@ public class XcosImageConverter implements ExternalImageConverter {
             return convertToPNG(name, schema.getAbsolutePath(), attributes, imageFile, imageName);
         } catch (Exception e) {
             e.printStackTrace();
-            System.err.printf("Problem when exporting Xcos schema %s !\n%s\n", schema, e.toString());
+            System.err.printf("Problem when exporting Xcos schema %s !%n%s%n", schema, e.toString());
         }
 
         return null;
index bb88456..bd08ffb 100644 (file)
@@ -2318,7 +2318,7 @@ public class XcosDiagram extends ScilabGraph {
         }
 
         if (GraphicsEnvironment.isHeadless()) {
-            System.err.printf("%s at %s\n    %s: %s\n", "warnCell", getRootDiagram().getTitle(), uid, message);
+            System.err.printf("%s at %s%n    %s: %s%n", "warnCell", getRootDiagram().getTitle(), uid, message);
             return;
         }