Author: bodewig
Date: Tue Aug 16 03:13:32 2011
New Revision: 1158092

URL: http://svn.apache.org/viewvc?rev=1158092&view=rev
Log:
reduce public API of dump package, fix equals in DumpArchiveSummary. 
COMPRESS-132

Modified:
    
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/Dirent.java
    
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveEntry.java
    
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java
    
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveSummary.java
    
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveUtil.java
    
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/TapeInputStream.java

Modified: 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/Dirent.java
URL: 
http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/Dirent.java?rev=1158092&r1=1158091&r2=1158092&view=diff
==============================================================================
--- 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/Dirent.java
 (original)
+++ 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/Dirent.java
 Tue Aug 16 03:13:32 2011
@@ -18,11 +18,10 @@
  */
 package org.apache.commons.compress.archivers.dump;
 
-
 /**
  * Directory entry.
  */
-public class Dirent {
+class Dirent {
     private int ino;
     private int parentIno;
     private int type;
@@ -36,7 +35,7 @@ public class Dirent {
      * @param type
      * @param name
      */
-    public Dirent(int ino, int parentIno, int type, String name) {
+    Dirent(int ino, int parentIno, int type, String name) {
         this.ino = ino;
         this.parentIno = parentIno;
         this.type = type;
@@ -47,7 +46,7 @@ public class Dirent {
      * Get ino.
      * @return the i-node
      */
-    public int getIno() {
+    int getIno() {
         return ino;
     }
 
@@ -55,7 +54,7 @@ public class Dirent {
      * Get ino of parent directory.
      * @return the parent i-node
      */
-    public int getParentIno() {
+    int getParentIno() {
         return parentIno;
     }
 
@@ -63,7 +62,7 @@ public class Dirent {
      * Get entry type.
      * @return the entry type
      */
-    public int getType() {
+    int getType() {
         return type;
     }
 
@@ -71,7 +70,7 @@ public class Dirent {
      * Get name of directory entry.
      * @return the directory name
      */
-    public String getName() {
+    String getName() {
         return name;
     }
 

Modified: 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveEntry.java
URL: 
http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveEntry.java?rev=1158092&r1=1158091&r2=1158092&view=diff
==============================================================================
--- 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveEntry.java
 (original)
+++ 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveEntry.java
 Tue Aug 16 03:13:32 2011
@@ -425,7 +425,7 @@ public class DumpArchiveEntry implements
      * @param buffer
      * @throws Exception
      */
-    public static DumpArchiveEntry parse(byte[] buffer) {
+    static DumpArchiveEntry parse(byte[] buffer) {
         DumpArchiveEntry entry = new DumpArchiveEntry();
         TapeSegmentHeader header = entry.header;
 
@@ -492,7 +492,7 @@ public class DumpArchiveEntry implements
     /**
      * Update entry with information from next tape segment header.
      */
-    public void update(byte[] buffer) {
+    void update(byte[] buffer) {
         header.volume = DumpArchiveUtil.convert32(buffer, 16);
         header.count = DumpArchiveUtil.convert32(buffer, 160);
 
@@ -511,7 +511,7 @@ public class DumpArchiveEntry implements
      * Archive entry as stored on tape. There is one TSH for (at most)
      * every 512k in the file.
      */
-    public static class TapeSegmentHeader {
+    static class TapeSegmentHeader {
         private DumpArchiveConstants.SEGMENT_TYPE type;
         private int volume;
         private int ino;

Modified: 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java
URL: 
http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java?rev=1158092&r1=1158091&r2=1158092&view=diff
==============================================================================
--- 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java
 (original)
+++ 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java
 Tue Aug 16 03:13:32 2011
@@ -127,7 +127,7 @@ public class DumpArchiveInputStream exte
     /**
      * Read CLRI (deleted inode) segment.
      */
-    public void readCLRI() throws IOException {
+    private void readCLRI() throws IOException {
         byte[] readBuf = raw.readRecord();
 
         if (!DumpArchiveUtil.verify(readBuf)) {
@@ -148,7 +148,7 @@ public class DumpArchiveInputStream exte
     /**
      * Read BITS segment.
      */
-    public void readBITS() throws IOException {
+    private void readBITS() throws IOException {
         byte[] readBuf = raw.readRecord();
 
         if (!DumpArchiveUtil.verify(readBuf)) {
@@ -264,7 +264,7 @@ public class DumpArchiveInputStream exte
     /**
      * Read directory entry.
      */
-    public void readDirectoryEntry(DumpArchiveEntry entry)
+    private void readDirectoryEntry(DumpArchiveEntry entry)
         throws IOException {
         long size = entry.getSize();
         boolean first = true;
@@ -351,7 +351,7 @@ public class DumpArchiveInputStream exte
      * @param entry
      * @return  full path for specified archive entry, or null if there's a 
gap.
      */
-    public String getPath(DumpArchiveEntry entry) {
+    private String getPath(DumpArchiveEntry entry) {
         // build the stack of elements. It's possible that we're 
         // still missing an intermediate value and if so we
         Stack<String> elements = new Stack<String>();

Modified: 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveSummary.java
URL: 
http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveSummary.java?rev=1158092&r1=1158091&r2=1158092&view=diff
==============================================================================
--- 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveSummary.java
 (original)
+++ 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveSummary.java
 Tue Aug 16 03:13:32 2011
@@ -41,7 +41,7 @@ public class DumpArchiveSummary {
     private int firstrec;
     private int ntrec;
 
-    public DumpArchiveSummary(byte[] buffer) {
+    DumpArchiveSummary(byte[] buffer) {
         dumpDate = new Date(1000L * DumpArchiveUtil.convert32(buffer, 4));
         previousDumpDate = new Date(1000L * DumpArchiveUtil.convert32(buffer, 
8));
         volume = DumpArchiveUtil.convert32(buffer, 12);
@@ -311,7 +311,7 @@ public class DumpArchiveSummary {
             return true;
         }
 
-        if (!(o instanceof DumpArchiveSummary)) {
+        if (!o.getClass().equals(getClass())) {
             return false;
         }
 

Modified: 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveUtil.java
URL: 
http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveUtil.java?rev=1158092&r1=1158091&r2=1158092&view=diff
==============================================================================
--- 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveUtil.java
 (original)
+++ 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveUtil.java
 Tue Aug 16 03:13:32 2011
@@ -22,7 +22,7 @@ package org.apache.commons.compress.arch
 /**
  * Various utilities for dump archives.
  */
-public class DumpArchiveUtil {
+class DumpArchiveUtil {
     /**
      * Private constructor to prevent instantiation.
      */

Modified: 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/TapeInputStream.java
URL: 
http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/TapeInputStream.java?rev=1158092&r1=1158091&r2=1158092&view=diff
==============================================================================
--- 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/TapeInputStream.java
 (original)
+++ 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/dump/TapeInputStream.java
 Tue Aug 16 03:13:32 2011
@@ -33,7 +33,7 @@ import java.util.zip.Inflater;
  *
  * @NotThreadSafe
  */
-public class TapeInputStream extends FilterInputStream {
+class TapeInputStream extends FilterInputStream {
     private byte[] blockBuffer = new byte[DumpArchiveConstants.TP_SIZE];
     private int currBlkIdx = -1;
     private int blockSize = DumpArchiveConstants.TP_SIZE;


Reply via email to