Javadoc, Checkstyle. More precise throw clause in private method.

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

Author: garydgregory

src/main/java/org/apache/commons/io/FileUtils.java

@@ -480,20 +480,26 @@ private static void validateListFilesParameters(File directory, IOFileFilter fil
         }
     }
 
+    /**
+     * Returns a filter that accepts files in addition to the {@link File} objects accepted by the given filter.
+     * 
+     * @param fileFilter a base filter to add to
+     * @return a filter that accepts files 
+     */
     private static IOFileFilter setUpEffectiveFileFilter(IOFileFilter fileFilter) {
         return FileFilterUtils.and(fileFilter, FileFilterUtils.notFileFilter(DirectoryFileFilter.INSTANCE));
     }
 
-	private static IOFileFilter setUpEffectiveDirFilter(IOFileFilter dirFilter) {
-		IOFileFilter effDirFilter;
-        if (dirFilter == null) {
-            effDirFilter = FalseFileFilter.INSTANCE;
-        } else {
-            effDirFilter = FileFilterUtils.and(dirFilter,
+    /**
+     * Returns a filter that accepts directories in addition to the {@link File} objects accepted by the given filter.
+     * 
+     * @param dirFilter a base filter to add to
+     * @return a filter that accepts directories 
+     */
+    private static IOFileFilter setUpEffectiveDirFilter(IOFileFilter dirFilter) {
+        return dirFilter == null ? FalseFileFilter.INSTANCE : FileFilterUtils.and(dirFilter,
                 DirectoryFileFilter.INSTANCE);
-        }
-        return effDirFilter;
-	}
+    }
 
     /**
      * Finds files within a given directory (and optionally its
@@ -571,10 +577,9 @@ public static Iterator<File> iterateFiles(
      * @see org.apache.commons.io.filefilter.FileFilterUtils
      * @see org.apache.commons.io.filefilter.NameFileFilter
      */
-	public static Iterator<File> iterateFilesAndDirs(
-			File directory,	IOFileFilter fileFilter, IOFileFilter dirFilter) {
-		return listFilesAndDirs(directory, fileFilter, dirFilter).iterator();
-	}
+    public static Iterator<File> iterateFilesAndDirs(File directory, IOFileFilter fileFilter, IOFileFilter dirFilter) {
+        return listFilesAndDirs(directory, fileFilter, dirFilter).iterator();
+    }
 
     //-----------------------------------------------------------------------
     /**

src/main/java/org/apache/commons/io/input/CharSequenceInputStream.java

@@ -21,6 +21,7 @@
 import java.io.InputStream;
 import java.nio.ByteBuffer;
 import java.nio.CharBuffer;
+import java.nio.charset.CharacterCodingException;
 import java.nio.charset.Charset;
 import java.nio.charset.CharsetEncoder;
 import java.nio.charset.CoderResult;
@@ -93,7 +94,13 @@ public CharSequenceInputStream(final CharSequence s, final String charset) {
         this(s, charset, 2048);
     }
 
-    private void fillBuffer() throws IOException {
+    /**
+     * Fills the byte output buffer from the input char buffer.
+     * 
+     * @throws CharacterCodingException
+     *             an error encoding data
+     */
+    private void fillBuffer() throws CharacterCodingException {
         this.bbuf.compact();
         CoderResult result = this.encoder.encode(this.cbuf, this.bbuf, true);
         if (result.isError()) {

src/main/java/org/apache/commons/io/input/ReaderInputStream.java

@@ -182,6 +182,12 @@ public ReaderInputStream(Reader reader) {
         this(reader, Charset.defaultCharset());
     }
 
+    /**
+     * Fills the internal char buffer from the reader.
+     * 
+     * @throws IOException
+     *             If an I/O error occurs
+     */
     private void fillBuffer() throws IOException {
         if (!endOfInput && (lastCoderResult == null || lastCoderResult.isUnderflow())) {
             encoderIn.compact();