Code clean up.

alexruiz · May 12, 2013 · 2b1aaef · 2b1aaef
1 parent 12e3b71
commit 2b1aaef
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 12 deletions.
diff --git a/src/main/java/org/fest/util/Files.java b/src/main/java/org/fest/util/Files.java
@@ -66,7 +66,7 @@ private Files() {
   private static @Nonnull List<String> fileNamesIn(@Nonnull File dir, boolean recurse) {
     List<String> fileNames = new ArrayList<String>();
     File[] existingFiles = dir.listFiles();
-    if (isNullOrEmpty(existingFiles)) {
+    if (existingFiles == null) {
       return fileNames;
     }
     for (File existingFile : existingFiles) {
@@ -85,8 +85,6 @@ private Files() {
   }
 
   /**
-   * Returns the system's temporary directory.
-   *
    * @return the system's temporary directory.
    * @throws IORuntimeException if this method cannot find or create the system's temporary directory.
    */

diff --git a/src/main/java/org/fest/util/IntrospectionError.java b/src/main/java/org/fest/util/IntrospectionError.java
@@ -24,8 +24,6 @@
  * @author Alex Ruiz
  */
 public class IntrospectionError extends RuntimeException {
-  private static final long serialVersionUID = 1L;
-
   /**
    * Creates a new {@link IntrospectionError}.
    *

diff --git a/src/main/java/org/fest/util/Sets.java b/src/main/java/org/fest/util/Sets.java
@@ -18,6 +18,7 @@
 import java.util.HashSet;
 import java.util.LinkedHashSet;
 
+import static java.util.Collections.addAll;
 import static org.fest.util.Preconditions.checkNotNull;
 
 /**
@@ -80,9 +81,7 @@ private Sets() {
   public static @Nonnull <T> LinkedHashSet<T> newLinkedHashSet(@Nonnull T... elements) {
     checkNotNull(elements);
     LinkedHashSet<T> set = new LinkedHashSet<T>();
-    for (T e : elements) {
-      set.add(e);
-    }
+    addAll(set, elements);
     return set;
   }
 }
diff --git a/src/main/javadocs/overview.html b/src/main/javadocs/overview.html
@@ -1,13 +1,14 @@
 <?xml version="1.0" encoding="UTF-8" ?>
-<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
+        "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml">
 <head>
-<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
+    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
 </head>
 <body>
 <p>
-This document is the API specification for the <a href="http://code.google.com/p/fest/" target="_blank">FEST</a> Utilities 
-Module 0.2-SNAPSHOT.
+    This document is the API specification for the <a href="http://code.google.com/p/fest/" target="_blank">FEST</a>
+    Utilities Module.
 </p>
 </body>
 </html>