Remove and enforce trailing whitespace

Also Javadoc author tags

Author: garydgregory

checkstyle.xml

@@ -43,7 +43,17 @@
   <!-- See http://checkstyle.sf.net/config_sizes.html -->
   <module name="FileLength" />
   <module name="FileTabCharacterCheck" />
-
+  <module name="RegexpSingleline">
+    <property name="format" value="^\s+\*\s+@author\s" />
+    <property name="message" value="Deprecated @author tag" />
+    <property name="fileExtensions" value="java" />
+    <property name="severity" value="warning" />
+  </module>
+  <module name="RegexpSingleline">
+    <!-- \s matches whitespace character, $ matches end of line. -->
+    <property name="format" value="\s+$"/>
+    <property name="message" value="Line has trailing spaces."/>
+  </module>
   <module name="TreeWalker">
     <module name="ExplicitInitialization" />
     <!--property name="cacheFile" value="${checkstyle.cache.file}"/ -->

src/main/java/org/apache/commons/jxpath/AbstractFactory.java

@@ -26,7 +26,7 @@
  */
 public abstract class AbstractFactory {
 
-    
+
     /**
      * Constructs a new instance for subclasses.
      */

src/main/java/org/apache/commons/jxpath/BasicNodeSet.java

@@ -31,7 +31,7 @@ public class BasicNodeSet implements NodeSet {
     private List<Pointer> readOnlyPointers;
     private List nodes;
     private List values;
-    
+
     /**
      * Constructs a new instance.
      */
@@ -41,7 +41,7 @@ public BasicNodeSet() {
 
     /**
      * Add the specified NodeSet to this NodeSet.
-     * 
+     *
      * @param nodeSet to add
      */
     public void add(final NodeSet nodeSet) {
@@ -52,7 +52,7 @@ public void add(final NodeSet nodeSet) {
 
     /**
      * Add a pointer to this NodeSet.
-     * 
+     *
      * @param pointer to add
      */
     public void add(final Pointer pointer) {
@@ -96,7 +96,7 @@ public synchronized List getValues() {
 
     /**
      * Remove a pointer from this NodeSet.
-     * 
+     *
      * @param pointer to remove
      */
     public void remove(final Pointer pointer) {

src/main/java/org/apache/commons/jxpath/BasicVariables.java

@@ -30,7 +30,7 @@ public class BasicVariables implements Variables {
      * Contains the values of declared variables
      */
     private final HashMap vars = new HashMap();
-    
+
     /**
      * Constructs a new instance.
      */

src/main/java/org/apache/commons/jxpath/ClassFunctions.java

@@ -30,11 +30,11 @@
  * Extension functions provided by a Java class.
  *
  * Let's say we declared a ClassFunction like this: <blockquote>
- * 
+ *
  * <pre>
  * new ClassFunctions(Integer.class, "int")
  * </pre>
- * 
+ *
  * </blockquote>
  *
  * We can now use XPaths like:
@@ -58,7 +58,7 @@ public class ClassFunctions implements Functions {
 
     /**
      * Constructs a new ClassFunctions.
-     * 
+     *
      * @param functionClass Class providing the functions
      * @param namespace     assigned ns
      */

src/main/java/org/apache/commons/jxpath/CompiledExpression.java

@@ -33,7 +33,7 @@ public interface CompiledExpression {
     /**
      * Creates intermediate elements of the path by invoking an {@link AbstractFactory}, which should first be installed on the context by calling
      * {@link JXPathContext#setFactory}.
-     * 
+     *
      * @param context base
      * @return Pointer created
      */
@@ -49,7 +49,7 @@ public interface CompiledExpression {
      * <li>The AbstractFactory fails to create an instance for an intermediate element.
      * <li>The property is not writable (no public, non-static set method)
      * </ul>
-     * 
+     *
      * @param context base
      * @param value   to set
      * @return Pointer created
@@ -59,7 +59,7 @@ public interface CompiledExpression {
     /**
      * Traverses the xpath and returns a Pointer. A Pointer provides easy access to a property. If the xpath matches no properties in the graph, the pointer
      * will be null.
-     * 
+     *
      * @param context base
      * @param xpath   string
      * @return Pointer found
@@ -68,15 +68,15 @@ public interface CompiledExpression {
 
     /**
      * Evaluates the xpath and returns the resulting object. Primitive types are wrapped into objects.
-     * 
+     *
      * @param context to evaluate
      * @return Object
      */
     Object getValue(JXPathContext context);
 
     /**
      * Evaluates the xpath, converts the result to the specified class and returns the resulting object.
-     * 
+     *
      * @param context      to evaluate
      * @param requiredType return type
      * @return Object
@@ -86,7 +86,7 @@ public interface CompiledExpression {
     /**
      * Traverses the xpath and returns a Iterator of all results found for the path. If the xpath matches no properties in the graph, the Iterator will not be
      * null.
-     * 
+     *
      * @param context base
      * @return Iterator
      */
@@ -95,22 +95,22 @@ public interface CompiledExpression {
     /**
      * Traverses the xpath and returns an Iterator of Pointers. A Pointer provides easy access to a property. If the xpath matches no properties in the graph,
      * the Iterator be empty, but not null.
-     * 
+     *
      * @param context to iterate
      * @return Iterator
      */
     Iterator iteratePointers(JXPathContext context);
 
     /**
      * Remove all graph elements described by this expression.
-     * 
+     *
      * @param context base
      */
     void removeAll(JXPathContext context);
 
     /**
      * Remove the graph element described by this expression.
-     * 
+     *
      * @param context base
      */
     void removePath(JXPathContext context);
@@ -121,7 +121,7 @@ public interface CompiledExpression {
      * <li>The xpath does not in fact describe an existing property
      * <li>The property is not writable (no public, non-static set method)
      * </ul>
-     * 
+     *
      * @param context base
      * @param value   to set
      */

src/main/java/org/apache/commons/jxpath/Container.java

@@ -27,14 +27,14 @@ public interface Container extends Serializable {
 
     /**
      * Returns the contained value.
-     * 
+     *
      * @return Object value
      */
     Object getValue();
 
     /**
      * Modifies the value contained by this container. May throw UnsupportedOperationException.
-     * 
+     *
      * @param value Object value to set.
      */
     void setValue(Object value);

src/main/java/org/apache/commons/jxpath/DynamicPropertyHandler.java

@@ -35,7 +35,7 @@ public interface DynamicPropertyHandler {
 
     /**
      * Returns the value of the specified dynamic property.
-     * 
+     *
      * @param object       to search
      * @param propertyName to retrieve
      * @return Object
@@ -44,15 +44,15 @@ public interface DynamicPropertyHandler {
 
     /**
      * Returns a list of dynamic property names for the supplied object.
-     * 
+     *
      * @param object to inspect
      * @return String[]
      */
     String[] getPropertyNames(Object object);
 
     /**
      * Modifies the value of the specified dynamic property.
-     * 
+     *
      * @param object       to modify
      * @param propertyName to modify
      * @param value        to set

src/main/java/org/apache/commons/jxpath/ExceptionHandler.java

@@ -19,14 +19,14 @@
 
 /**
  * Exception handler interface. Actually handles Throwables.
- * 
+ *
  * @since 1.4
  */
 public interface ExceptionHandler {
 
     /**
      * Handle an encountered Throwable.
-     * 
+     *
      * @param t   to handle
      * @param ptr specific context
      */

src/main/java/org/apache/commons/jxpath/ExpressionContext.java

@@ -23,10 +23,10 @@
  * If an extenstion function has an argument of type ExpressionContext, it can gain access to the current node of an XPath expression context.
  * <p>
  * Example: <blockquote>
- * 
+ *
  * <pre>
  * public class MyExtenstionFunctions {
- * 
+ *
  *     public static String objectType(ExpressionContext context) {
  *         Object value = context.getContextNodePointer().getValue();
  *         if (value == null) {
@@ -36,15 +36,15 @@
  *     }
  * }
  * </pre>
- * 
+ *
  * </blockquote>
  *
  * You can then register this extension function using a {@link ClassFunctions ClassFunctions} object and call it like this: <blockquote>
- * 
+ *
  * <pre>
  * "/descendent-or-self::node()[ns:objectType() = 'java.util.Date']"
  * </pre>
- * 
+ *
  * </blockquote> This expression will find all nodes of the graph that are dates.
  */
 public interface ExpressionContext {
@@ -72,7 +72,7 @@ public interface ExpressionContext {
 
     /**
      * Returns the current context position.
-     * 
+     *
      * @return int
      */
     int getPosition();