Index: /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Compatibility.java
===================================================================
--- /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Compatibility.java	(revision 3422)
+++ /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Compatibility.java	(revision 3423)
@@ -34,9 +34,9 @@
 
 	/**
-	 * Checks if a browser's user agent string is compatible for the FCKeditor.
+	 * Checks if a browser's user agent string is compatible with the FCKeditor.
 	 * 
 	 * @param userAgentString
-	 *            user agent string of a browser
-	 * @return <code>true</code> if compatible otherwise <code>false</code>
+	 *            the user agent string of a browser
+	 * @return <code>true</code> if compatible, else <code>false</code>
 	 */
 	public static boolean check(final String userAgentString) {
@@ -71,10 +71,12 @@
 
 	/**
-	 * Checks browser compatibility by reading its user-agent header from the
-	 * request.
+	 * Returns <code>true</code> if a browser is compatible by its request
+	 * user-agent header.
 	 * 
 	 * @see #check(String)
 	 * @param request
 	 *            current user request instance
+	 * @return <code>true</code> if a browser is compatible, else
+	 *         <code>false</code>
 	 */
 	public static boolean isCompatibleBrowser(final HttpServletRequest request) {
@@ -87,12 +89,14 @@
 	 * first matching group has to be the version number!
 	 * 
-	 * @param userAgent
+	 * @param userAgentString
+	 *            the user agent string of a browser
 	 * @param regex
-	 * @return The browser version, or -1f if version can't be determined.
+	 *            the pattern to retrieve the browser version
+	 * @return the browser version, or -1f if version can't be determined
 	 */
-	private static float getBrowserVersion(final String userAgent,
+	private static float getBrowserVersion(final String userAgentString,
 			final String regex) {
 		Pattern pattern = Pattern.compile(regex);
-		Matcher matcher = pattern.matcher(userAgent);
+		Matcher matcher = pattern.matcher(userAgentString);
 		if (matcher.matches()) {
 			try {
Index: /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Utils.java
===================================================================
--- /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Utils.java	(revision 3422)
+++ /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Utils.java	(revision 3423)
@@ -37,10 +37,10 @@
 	 * 
 	 * @param stringList
-	 *            string separated with a delimiter
+	 *            strings separated with a delimiter
 	 * @param delimiter
 	 *            separating delimiter
 	 * @return a lower-cased set, empty set if stringList is empty
 	 * @throws IllegalArgumentException
-	 *             if 'delimiter' is empty.
+	 *             if <code>delimiter</code> is empty
 	 */
 	public static Set<String> getSet(final String stringList,
@@ -68,8 +68,8 @@
 	 * @see #getSet(String, String)
 	 * @param stringList
-	 *            string separated with a delimiter
+	 *            strings separated with a delimiter
 	 * @return a lower-cased set, empty set if stringList is empty
 	 * @throws IllegalArgumentException
-	 *             if 'delimiter' is empty.
+	 *             if <code>delimiter</code> is empty
 	 */
 	public static Set<String> getSet(final String stringList) {
@@ -78,9 +78,10 @@
 
 	/**
-	 * Checks if a string is <code>null</code> or empty (length zero).
+	 * Checks if a string is empty ("") or null.
 	 * 
 	 * @param str
-	 *            string to check
-	 * @return <code>true</code> if the string is <code>null</code> or empty.
+	 *            string to check, may be null
+	 * @return <code>true</code> if the string is <code>null</code> or empty,
+	 *         else <code>false</code>
 	 */
 	public static boolean isEmpty(final String str) {
@@ -89,10 +90,10 @@
 
 	/**
-	 * Inverts return value of {@link #isEmpty(String)}.
+	 * Checks if a string is not empty ("") and not null.
 	 * 
 	 * @param str
-	 *            string to check
-	 * @return <code>true</code> if the string is not empty or not
-	 *         <code>null</code>.
+	 *            string to check, may be null
+	 * @return <code>true</code> if the string is not empty and not
+	 *         <code>null</code>, else <code>false</code>
 	 */
 	public static boolean isNotEmpty(final String str) {
@@ -101,12 +102,11 @@
 
 	/**
-	 * Checks if a string is empty or contains whitespace only.<br />
-	 * This method calls {@link #isEmpty(String)} first and then checks the
-	 * string with {@link Character#isWhitespace(char)}.
+	 * Checks if a string is whitespace, empty ("") or null. Whitespace is check
+	 * by {@link Character#isWhitespace(char)}.
 	 * 
 	 * @param str
-	 *            string to check
+	 *            string to check, may be null
 	 * @return <code>true</code> if the string is <code>null</code>, empty or
-	 *         contains whitespace only.
+	 *         whitespace
 	 */
 	public static boolean isBlank(final String str) {
@@ -124,10 +124,10 @@
 
 	/**
-	 * Inverts return value of {@link #isBlank(String)}.
+	 * Checks if a string is not empty (""), not null and not whitespace.
 	 * 
 	 * @param str
-	 *            string to check
+	 *            string to check, may be null
 	 * @return <code>true</code> if the string is not <code>null</code>, not
-	 *         empty or does not contain whitespace only.
+	 *         empty and not whitespace.
 	 */
 	public static boolean isNotBlank(final String str) {
Index: /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/UtilsFile.java
===================================================================
--- /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/UtilsFile.java	(revision 3422)
+++ /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/UtilsFile.java	(revision 3423)
@@ -85,5 +85,5 @@
 	 *            input stream of an image
 	 * @return <code>true</code> if the underlying input stream contains an
-	 *         image else <code>false</code>.
+	 *         image, else <code>false</code>
 	 */
 	public static boolean isImage(final InputStream in) {
@@ -100,6 +100,6 @@
 	 * @param path
 	 *            a potentially 'malicious' path
-	 * @return <code>true</code> if path complies to the rules else
-	 *         <code>false</code>.
+	 * @return <code>true</code> if path complies to the rules, else
+	 *         <code>false</code>
 	 */
 	public static boolean isValidPath(final String path) {
@@ -133,5 +133,6 @@
 	 * 
 	 * @param filename
-	 * @return <code>true</code> if filename contains severals dots else
+	 *            filename to check
+	 * @return <code>true</code> if filename contains severals dots, else
 	 *         <code>false</code>
 	 */
@@ -144,5 +145,5 @@
 	 * 
 	 * @param dir
-	 *            Directory to check/create.
+	 *            directory to check/create
 	 */
 	public static void checkDirAndCreate(File dir) {
Index: /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/UtilsResponse.java
===================================================================
--- /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/UtilsResponse.java	(revision 3422)
+++ /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/UtilsResponse.java	(revision 3423)
@@ -47,11 +47,11 @@
 	 * 
 	 * @param userFilesPath
-	 *            current userfiles path (may be null)
+	 *            the current userfiles path (may be null)
 	 * @param type
-	 *            current resource type
+	 *            the current resource type
 	 * @param currentFolder
-	 *            selected current folder
+	 *            the selected current folder
 	 * @param filename
-	 *            current chosen file (may be null)
+	 *            the current chosen file (may be null)
 	 * @return assembled url for the File Browser
 	 */
Index: /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/XHtmlTagTool.java
===================================================================
--- /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/XHtmlTagTool.java	(revision 3422)
+++ /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/XHtmlTagTool.java	(revision 3423)
@@ -25,8 +25,7 @@
 
 /**
- * Java representation of an XHTML tag.<br />
- * <br>
+ * Java representation of an XHTML tag.
+ * <p>
  * Usage:
- * 
  * <pre>
  * XHtmlTagTool tag = XHtmlTagTool(&quot;a&quot;, &quot;link&quot;);
@@ -34,5 +33,5 @@
  * tag.toString();: &lt;a href=&quot;http://google.com&quot;&gt;link&lt;/a&gt;
  * </pre>
- * 
+ * </p>
  * <em>Note</em>:
  * <ul>
@@ -62,7 +61,7 @@
 	 * 
 	 * @param name
-	 *            tag name
+	 *            the tag name of the new XHtmlTagTool
 	 * @throws IllegalArgumentException
-	 *             if name is empty
+	 *             if <code>name</code> is empty
 	 */
 	public XHtmlTagTool(final String name) {
@@ -77,9 +76,9 @@
 	 * 
 	 * @param name
-	 *            tag name
+	 *             the tag name of the new XHtmlTagTool
 	 * @param value
-	 *            tag value which is the tag body
+	 *            the tag value of the new XHtmlTagTool which is the tag body
 	 * @throws IllegalArgumentException
-	 *             if name is empty
+	 *             if <code>name</code> is empty
 	 */
 	public XHtmlTagTool(final String name, final String value) {
@@ -92,5 +91,5 @@
 	 * 
 	 * @param value
-	 *            tag value which is the tag body
+	 *            the tag value which is the tag body
 	 */
 	public void setValue(final String value) {
@@ -106,10 +105,9 @@
 	 *            attribute value
 	 * @throws IllegalArgumentException
-	 *             if 'name' is empty.
+	 *             if <code>name</code> is empty
 	 */
 	public void addAttribute(final String name, final String value) {
 		if (Utils.isEmpty(name))
-			throw new IllegalArgumentException(
-					"Parameter 'name' shouldn't be empty!");
+			throw new IllegalArgumentException("Name shouldn't be empty");
 		attributes.put(name, value);
 	}
@@ -141,27 +139,3 @@
 	}
 
-	@Override
-	public boolean equals(Object obj) {
-		if (obj == null)
-			return false;
-		try {
-			XHtmlTagTool tag = (XHtmlTagTool) obj;
-			// FIXME potential NPE on value
-			return value.equals(tag.value) && name.equals(tag.name)
-					&& attributes.equals(tag.attributes);
-		} catch (ClassCastException e) {
-			return false;
-		}
-	}
-
-	/**
-	 * The hash code is merely a sum of name's, value's, and attributes' hash
-	 * code.
-	 */
-	@Override
-	public int hashCode() {
-
-		// FIXME potential NPE on value
-		return name.hashCode() + value.hashCode() + attributes.hashCode();
-	}
 }