commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bode...@apache.org
Subject [1/3] commons-compress git commit: move NOSONAR to first line of multi-line statements
Date Tue, 20 Dec 2016 13:41:38 GMT
Repository: commons-compress
Updated Branches:
  refs/heads/master 066ee4cd3 -> 5e0980f01


move NOSONAR to first line of multi-line statements


Project: http://git-wip-us.apache.org/repos/asf/commons-compress/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-compress/commit/a0336212
Tree: http://git-wip-us.apache.org/repos/asf/commons-compress/tree/a0336212
Diff: http://git-wip-us.apache.org/repos/asf/commons-compress/diff/a0336212

Branch: refs/heads/master
Commit: a03362129bd63208aa6c0a5a47d5ddd2b0cc81f1
Parents: 066ee4c
Author: Stefan Bodewig <bodewig@apache.org>
Authored: Tue Dec 20 14:36:10 2016 +0100
Committer: Stefan Bodewig <bodewig@apache.org>
Committed: Tue Dec 20 14:36:10 2016 +0100

----------------------------------------------------------------------
 .../apache/commons/compress/archivers/sevenz/SevenZFile.java | 4 ++--
 .../compress/archivers/tar/TarArchiveOutputStream.java       | 8 ++++----
 .../commons/compress/archivers/zip/ZipArchiveEntry.java      | 4 ++--
 3 files changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-compress/blob/a0336212/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java b/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
index 9a14af6..874885e 100644
--- a/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
+++ b/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
@@ -357,8 +357,8 @@ public class SevenZFile implements Closeable {
             if (coder.numInStreams != 1 || coder.numOutStreams != 1) {
                 throw new IOException("Multi input/output stream coders are not yet supported");
             }
-            inputStreamStack = Coders.addDecoder(fileName, inputStreamStack,
-                    folder.getUnpackSizeForCoder(coder), coder, password); //NOSONAR
+            inputStreamStack = Coders.addDecoder(fileName, inputStreamStack, //NOSONAR
+                    folder.getUnpackSizeForCoder(coder), coder, password);
         }
         if (folder.hasCrc) {
             inputStreamStack = new CRC32VerifyingInputStream(inputStreamStack,

http://git-wip-us.apache.org/repos/asf/commons-compress/blob/a0336212/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
b/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
index 8c44a07..79c7bca 100644
--- a/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
+++ b/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
@@ -633,9 +633,9 @@ public class TarArchiveOutputStream extends ArchiveOutputStream {
 
     private void failForBigNumber(final String field, final long value, final long maxValue,
final String additionalMsg) {
         if (value < 0 || value > maxValue) {
-            throw new RuntimeException(field + " '" + value
+            throw new RuntimeException(field + " '" + value //NOSONAR
                     + "' is too big ( > "
-                    + maxValue + " )." + additionalMsg); //NOSONAR
+                    + maxValue + " )." + additionalMsg);
         }
     }
 
@@ -684,9 +684,9 @@ public class TarArchiveOutputStream extends ArchiveOutputStream {
                 write(0); // NUL terminator
                 closeArchiveEntry();
             } else if (longFileMode != LONGFILE_TRUNCATE) {
-                throw new RuntimeException(fieldName + " '" + name
+                throw new RuntimeException(fieldName + " '" + name //NOSONAR
                                            + "' is too long ( > "
-                                           + TarConstants.NAMELEN + " bytes)"); //NOSONAR
+                                           + TarConstants.NAMELEN + " bytes)");
             }
         }
         return false;

http://git-wip-us.apache.org/repos/asf/commons-compress/blob/a0336212/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveEntry.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveEntry.java
b/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveEntry.java
index bc5926e..5030da5 100644
--- a/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveEntry.java
+++ b/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveEntry.java
@@ -540,8 +540,8 @@ public class ZipArchiveEntry extends java.util.zip.ZipEntry
             mergeExtraFields(local, true);
         } catch (final ZipException e) {
             // actually this is not possible as of Commons Compress 1.1
-            throw new RuntimeException("Error parsing extra fields for entry: "
-                                       + getName() + " - " + e.getMessage(), e); //NOSONAR
+            throw new RuntimeException("Error parsing extra fields for entry: " //NOSONAR
+                                       + getName() + " - " + e.getMessage(), e);
         }
     }
 


Mime
View raw message