CODEC-138 Complete FilterInputStream interface for BaseNCodecInputStream

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/codec/trunk@1308606 13f79535-47bb-0310-9956-ffa450edef68

Author: sebbASF

src/changes/changes.xml

@@ -31,6 +31,9 @@
     </release>
     -->
     <release version="1.7" date="TBD" description="Feature and fix release.">
+      <action dev="sebb" type="fix" issue="CODEC-138">
+        Complete FilterInputStream interface for BaseNCodecInputStream
+      </action>
       <action dev="ggregory" type="fix" issue="CODEC-136">
         Use Charset objects when possible, create Charsets for required character encodings
       </action>

src/main/java/org/apache/commons/codec/binary/BaseNCodecInputStream.java

@@ -59,15 +59,26 @@ public int available() throws IOException {
         return baseNCodec.eof ? 0 : 1;
     }
 
+    /**
+     * Marks the current position in this input stream.
+     * <p>The {@link #mark} method of {@link BaseNCodecInputStream} does nothing.</p>
+     *
+     * @param readLimit the maximum limit of bytes that can be read before the mark position becomes invalid.
+     */
+    @Override
+    public synchronized void mark(int readLimit) {
+    }
+
     /**
      * {@inheritDoc}
-     * 
-     * @return false
+     *
+     * @return always returns <code>false</code>
      */
     @Override
     public boolean markSupported() {
         return false; // not an easy job to support marks
     }
+
     /**
      * Reads one <code>byte</code> from this input stream.
      * 
@@ -151,6 +162,17 @@ public int read(byte b[], int offset, int len) throws IOException {
         }
     }
 
+    /**
+     * Repositions this stream to the position at the time the mark method was last called on this input stream.
+     * <p>The {@link #reset} method of {@link BaseNCodecInputStream} does nothing except throw an {@link IOException}.</p>
+     *
+     * @throws IOException if this method is invoked
+     */
+    @Override
+    public synchronized void reset() throws IOException {
+        throw new IOException("mark/reset not supported");
+    }
+
     /**
      * {@inheritDoc}
      *