This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-jxpath.git

commit 6f371f9e82223024ce441518148841edcd8d1a6c
Author: Gary D. Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Mar 19 22:56:48 2025 -0400

    Javadoc
---
 .../org/apache/commons/jxpath/ClassFunctions.java  |   2 +-
 .../jxpath/JXPathAbstractFactoryException.java     |   2 +-
 .../apache/commons/jxpath/JXPathBasicBeanInfo.java |   6 +-
 .../commons/jxpath/JXPathContextFactory.java       |   2 +-
 .../JXPathContextFactoryConfigurationError.java    |   8 +-
 .../org/apache/commons/jxpath/JXPathException.java |   8 +-
 .../jxpath/JXPathFunctionNotFoundException.java    |   2 +-
 .../jxpath/JXPathInvalidAccessException.java       |   4 +-
 .../jxpath/JXPathInvalidSyntaxException.java       |   2 +-
 .../commons/jxpath/JXPathNotFoundException.java    |   2 +-
 .../jxpath/JXPathTypeConversionException.java      |   4 +-
 .../apache/commons/jxpath/PackageFunctions.java    |   2 +-
 .../commons/jxpath/XMLDocumentContainer.java       |  24 +++-
 .../jxpath/functions/ConstructorFunction.java      |   2 +-
 .../commons/jxpath/functions/MethodFunction.java   |   2 +-
 .../org/apache/commons/jxpath/ri/EvalContext.java  |   2 +-
 .../jxpath/ri/JXPathCompiledExpression.java        |   2 +-
 .../ri/JXPathContextFactoryReferenceImpl.java      |   2 +-
 .../jxpath/ri/JXPathContextReferenceImpl.java      |   4 +-
 .../commons/jxpath/ri/NamespaceResolver.java       |  31 +++--
 .../java/org/apache/commons/jxpath/ri/QName.java   |  24 +++-
 .../commons/jxpath/ri/axes/AncestorContext.java    |   2 +-
 .../commons/jxpath/ri/axes/AttributeContext.java   |   2 +-
 .../commons/jxpath/ri/axes/ChildContext.java       |   2 +-
 .../commons/jxpath/ri/axes/DescendantContext.java  |   2 +-
 .../commons/jxpath/ri/axes/InitialContext.java     |   2 +-
 .../commons/jxpath/ri/axes/NodeSetContext.java     |   2 +-
 .../commons/jxpath/ri/axes/ParentContext.java      |   2 +-
 .../ri/axes/PrecedingOrFollowingContext.java       |   2 +-
 .../commons/jxpath/ri/axes/PredicateContext.java   |   2 +-
 .../apache/commons/jxpath/ri/axes/RootContext.java |   2 +-
 .../apache/commons/jxpath/ri/axes/SelfContext.java |   2 +-
 .../commons/jxpath/ri/axes/UnionContext.java       |   2 +-
 .../commons/jxpath/ri/compiler/Constant.java       |   4 +-
 .../commons/jxpath/ri/compiler/CoreFunction.java   |   2 +-
 .../commons/jxpath/ri/compiler/CoreOperation.java  |   2 +-
 .../jxpath/ri/compiler/CoreOperationAdd.java       |   2 +-
 .../jxpath/ri/compiler/CoreOperationAnd.java       |   2 +-
 .../jxpath/ri/compiler/CoreOperationCompare.java   |   4 +-
 .../jxpath/ri/compiler/CoreOperationDivide.java    |   2 +-
 .../jxpath/ri/compiler/CoreOperationEqual.java     |   2 +-
 .../ri/compiler/CoreOperationGreaterThan.java      |   2 +-
 .../compiler/CoreOperationGreaterThanOrEqual.java  |   2 +-
 .../jxpath/ri/compiler/CoreOperationLessThan.java  |   2 +-
 .../ri/compiler/CoreOperationLessThanOrEqual.java  |   2 +-
 .../jxpath/ri/compiler/CoreOperationMod.java       |   2 +-
 .../jxpath/ri/compiler/CoreOperationMultiply.java  |   2 +-
 .../jxpath/ri/compiler/CoreOperationNegate.java    |   2 +-
 .../jxpath/ri/compiler/CoreOperationNotEqual.java  |   2 +-
 .../jxpath/ri/compiler/CoreOperationOr.java        |   2 +-
 .../CoreOperationRelationalExpression.java         |   2 +-
 .../jxpath/ri/compiler/CoreOperationSubtract.java  |   2 +-
 .../jxpath/ri/compiler/CoreOperationUnion.java     |   2 +-
 .../commons/jxpath/ri/compiler/Expression.java     |   4 +-
 .../commons/jxpath/ri/compiler/ExpressionPath.java |   2 +-
 .../jxpath/ri/compiler/ExtensionFunction.java      |   2 +-
 .../commons/jxpath/ri/compiler/LocationPath.java   |   2 +-
 .../jxpath/ri/compiler/NameAttributeTest.java      |   2 +-
 .../commons/jxpath/ri/compiler/NodeNameTest.java   |   4 +-
 .../commons/jxpath/ri/compiler/NodeTypeTest.java   |   2 +-
 .../commons/jxpath/ri/compiler/Operation.java      |   2 +-
 .../apache/commons/jxpath/ri/compiler/Path.java    |   2 +-
 .../ri/compiler/ProcessingInstructionTest.java     |   2 +-
 .../apache/commons/jxpath/ri/compiler/Step.java    |   2 +-
 .../jxpath/ri/compiler/VariableReference.java      |   2 +-
 .../commons/jxpath/ri/model/NodePointer.java       | 140 ++++++++++++---------
 .../commons/jxpath/ri/model/VariablePointer.java   |  28 ++++-
 .../jxpath/ri/model/VariablePointerFactory.java    |   2 +-
 .../ri/model/beans/BeanAttributeIterator.java      |   2 +-
 .../commons/jxpath/ri/model/beans/BeanPointer.java |  16 ++-
 .../jxpath/ri/model/beans/BeanPropertyPointer.java |   2 +-
 .../beans/CollectionAttributeNodeIterator.java     |   2 +-
 .../model/beans/CollectionChildNodeIterator.java   |   2 +-
 .../ri/model/beans/CollectionNodeIterator.java     |   2 +-
 .../jxpath/ri/model/beans/CollectionPointer.java   |   4 +-
 .../ri/model/beans/LangAttributePointer.java       |   2 +-
 .../jxpath/ri/model/beans/NullElementPointer.java  |   2 +-
 .../commons/jxpath/ri/model/beans/NullPointer.java |   4 +-
 .../jxpath/ri/model/beans/NullPropertyPointer.java |   2 +-
 .../jxpath/ri/model/beans/PropertyIterator.java    |   2 +-
 .../ri/model/beans/PropertyOwnerPointer.java       |   4 +-
 .../ri/model/container/ContainerPointer.java       |   4 +-
 .../jxpath/ri/model/dom/DOMAttributeIterator.java  |   2 +-
 .../jxpath/ri/model/dom/DOMAttributePointer.java   |   2 +-
 .../jxpath/ri/model/dom/DOMNamespaceIterator.java  |   2 +-
 .../jxpath/ri/model/dom/DOMNodeIterator.java       |   2 +-
 .../jxpath/ri/model/dom/DOMNodePointer.java        |   6 +-
 .../jxpath/ri/model/dom/NamespacePointer.java      |   4 +-
 .../jxpath/ri/model/dynabeans/DynaBeanPointer.java |   4 +-
 .../model/dynabeans/DynaBeanPropertyPointer.java   |   2 +-
 .../StrictLazyDynaBeanPointerFactory.java          |   4 +-
 .../ri/model/dynamic/DynamicAttributeIterator.java |   2 +-
 .../jxpath/ri/model/dynamic/DynamicPointer.java    |   4 +-
 .../ri/model/dynamic/DynamicPropertyIterator.java  |   2 +-
 .../ri/model/dynamic/DynamicPropertyPointer.java   |   2 +-
 .../ri/model/jdom/JDOMAttributeIterator.java       |   2 +-
 .../ri/model/jdom/JDOMNamespaceIterator.java       |   2 +-
 .../jxpath/ri/model/jdom/JDOMNamespacePointer.java |   4 +-
 .../jxpath/ri/model/jdom/JDOMNodeIterator.java     |   2 +-
 .../jxpath/ri/model/jdom/JDOMNodePointer.java      |   6 +-
 .../servlet/HttpSessionAndServletContext.java      |   2 +-
 .../commons/jxpath/servlet/KeywordVariables.java   |   2 +-
 .../commons/jxpath/servlet/PageScopeContext.java   |   2 +-
 .../jxpath/servlet/ServletRequestAndContext.java   |   2 +-
 .../commons/jxpath/util/BasicTypeConverter.java    |   4 +-
 .../commons/jxpath/util/KeyManagerUtils.java       |   2 +-
 .../commons/jxpath/util/ReverseComparator.java     |   2 +-
 107 files changed, 300 insertions(+), 223 deletions(-)

diff --git a/src/main/java/org/apache/commons/jxpath/ClassFunctions.java 
b/src/main/java/org/apache/commons/jxpath/ClassFunctions.java
index d5c9cf6..265a309 100644
--- a/src/main/java/org/apache/commons/jxpath/ClassFunctions.java
+++ b/src/main/java/org/apache/commons/jxpath/ClassFunctions.java
@@ -57,7 +57,7 @@ public class ClassFunctions implements Functions {
     private final String namespace;
 
     /**
-     * Create a new ClassFunctions.
+     * Constructs a new ClassFunctions.
      * 
      * @param functionClass Class providing the functions
      * @param namespace     assigned ns
diff --git 
a/src/main/java/org/apache/commons/jxpath/JXPathAbstractFactoryException.java 
b/src/main/java/org/apache/commons/jxpath/JXPathAbstractFactoryException.java
index 30cda39..8ccb779 100644
--- 
a/src/main/java/org/apache/commons/jxpath/JXPathAbstractFactoryException.java
+++ 
b/src/main/java/org/apache/commons/jxpath/JXPathAbstractFactoryException.java
@@ -25,7 +25,7 @@ public class JXPathAbstractFactoryException extends 
JXPathException {
     private static final long serialVersionUID = -4403564377958943239L;
 
     /**
-     * Create a new JXPathAbstractFactoryException.
+     * Constructs a new JXPathAbstractFactoryException.
      * 
      * @param message exception message
      */
diff --git a/src/main/java/org/apache/commons/jxpath/JXPathBasicBeanInfo.java 
b/src/main/java/org/apache/commons/jxpath/JXPathBasicBeanInfo.java
index 9084c96..c1b58e5 100644
--- a/src/main/java/org/apache/commons/jxpath/JXPathBasicBeanInfo.java
+++ b/src/main/java/org/apache/commons/jxpath/JXPathBasicBeanInfo.java
@@ -64,7 +64,7 @@ public class JXPathBasicBeanInfo implements JXPathBeanInfo {
     private transient HashMap propertyDescriptorMap;
 
     /**
-     * Create a new JXPathBasicBeanInfo.
+     * Constructs a new JXPathBasicBeanInfo.
      *
      * @param clazz bean class
      */
@@ -73,7 +73,7 @@ public class JXPathBasicBeanInfo implements JXPathBeanInfo {
     }
 
     /**
-     * Create a new JXPathBasicBeanInfo.
+     * Constructs a new JXPathBasicBeanInfo.
      *
      * @param clazz  bean class
      * @param atomic whether objects of this class are treated as atomic 
objects which have no properties of their own.
@@ -84,7 +84,7 @@ public class JXPathBasicBeanInfo implements JXPathBeanInfo {
     }
 
     /**
-     * Create a new JXPathBasicBeanInfo.
+     * Constructs a new JXPathBasicBeanInfo.
      *
      * @param clazz                       bean class
      * @param dynamicPropertyHandlerClass dynamic property handler class
diff --git a/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java 
b/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java
index 26b80f7..c999848 100644
--- a/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java
+++ b/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java
@@ -184,7 +184,7 @@ public abstract class JXPathContextFactory {
     }
 
     /**
-     * Create a new JXPathContextFactory.
+     * Constructs a new JXPathContextFactory.
      */
     protected JXPathContextFactory() {
     }
diff --git 
a/src/main/java/org/apache/commons/jxpath/JXPathContextFactoryConfigurationError.java
 
b/src/main/java/org/apache/commons/jxpath/JXPathContextFactoryConfigurationError.java
index 4dadc25..de55cb1 100644
--- 
a/src/main/java/org/apache/commons/jxpath/JXPathContextFactoryConfigurationError.java
+++ 
b/src/main/java/org/apache/commons/jxpath/JXPathContextFactoryConfigurationError.java
@@ -26,13 +26,13 @@ public class JXPathContextFactoryConfigurationError extends 
Error {
     private static final long serialVersionUID = 2L;
 
     /**
-     * Create a new {@code JXPathContextFactoryConfigurationError} with no 
detail mesage.
+     * Constructs a new {@code JXPathContextFactoryConfigurationError} with no 
detail mesage.
      */
     public JXPathContextFactoryConfigurationError() {
     }
 
     /**
-     * Create a new {@code JXPathContextFactoryConfigurationError} with a 
given {@code Exception} base cause of the error.
+     * Constructs a new {@code JXPathContextFactoryConfigurationError} with a 
given {@code Exception} base cause of the error.
      *
      * @param cause The exception to be encapsulated in a 
JXPathContextFactoryConfigurationError.
      */
@@ -41,7 +41,7 @@ public class JXPathContextFactoryConfigurationError extends 
Error {
     }
 
     /**
-     * Create a new {@code JXPathContextFactoryConfigurationError} with the 
given {@code Exception} base cause and detail message.
+     * Constructs a new {@code JXPathContextFactoryConfigurationError} with 
the given {@code Exception} base cause and detail message.
      *
      * @param cause   The exception to be encapsulated in a 
JXPathContextFactoryConfigurationError
      * @param msg The detail message.
@@ -51,7 +51,7 @@ public class JXPathContextFactoryConfigurationError extends 
Error {
     }
 
     /**
-     * Create a new {@code JXPathContextFactoryConfigurationError} with the 
{@code String } specified as an error message.
+     * Constructs a new {@code JXPathContextFactoryConfigurationError} with 
the {@code String } specified as an error message.
      *
      * @param msg The error message for the exception.
      */
diff --git a/src/main/java/org/apache/commons/jxpath/JXPathException.java 
b/src/main/java/org/apache/commons/jxpath/JXPathException.java
index 7ad18b2..3790497 100644
--- a/src/main/java/org/apache/commons/jxpath/JXPathException.java
+++ b/src/main/java/org/apache/commons/jxpath/JXPathException.java
@@ -25,13 +25,13 @@ public class JXPathException extends RuntimeException {
     private static final long serialVersionUID = 2L;
 
     /**
-     * Create a new {@code JXPathException} with no detail mesage.
+     * Constructs a new {@code JXPathException} with no detail mesage.
      */
     public JXPathException() {
     }
 
     /**
-     * Create a new {@code JXPathException} with the {@code String } specified 
as an error message.
+     * Constructs a new {@code JXPathException} with the {@code String } 
specified as an error message.
      *
      * @param msg The error message for the exception.
      */
@@ -40,7 +40,7 @@ public class JXPathException extends RuntimeException {
     }
 
     /**
-     * Create a new {@code JXPathException} with the given {@code Exception} 
base cause and detail message.
+     * Constructs a new {@code JXPathException} with the given {@code 
Exception} base cause and detail message.
      *
      * @param msg The detail message.
      * @param cause   The exception to be encapsulated in a JXPathException
@@ -50,7 +50,7 @@ public class JXPathException extends RuntimeException {
     }
 
     /**
-     * Create a new {@code JXPathException} with a given {@code Throwable} 
base cause of the error.
+     * Constructs a new {@code JXPathException} with a given {@code Throwable} 
base cause of the error.
      *
      * @param cause The exception to be encapsulated in a JXPathException.
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/JXPathFunctionNotFoundException.java 
b/src/main/java/org/apache/commons/jxpath/JXPathFunctionNotFoundException.java
index f956697..a225ba9 100644
--- 
a/src/main/java/org/apache/commons/jxpath/JXPathFunctionNotFoundException.java
+++ 
b/src/main/java/org/apache/commons/jxpath/JXPathFunctionNotFoundException.java
@@ -25,7 +25,7 @@ public class JXPathFunctionNotFoundException extends 
JXPathException {
     private static final long serialVersionUID = -8875537628056117241L;
 
     /**
-     * Create a new JXPathFunctionNotFoundException.
+     * Constructs a new JXPathFunctionNotFoundException.
      * 
      * @param message exception message
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/JXPathInvalidAccessException.java 
b/src/main/java/org/apache/commons/jxpath/JXPathInvalidAccessException.java
index 5e917c2..f0d6bc5 100644
--- a/src/main/java/org/apache/commons/jxpath/JXPathInvalidAccessException.java
+++ b/src/main/java/org/apache/commons/jxpath/JXPathInvalidAccessException.java
@@ -26,7 +26,7 @@ public class JXPathInvalidAccessException extends 
JXPathException {
     private static final long serialVersionUID = -8875537628056117241L;
 
     /**
-     * Create a new JXPathInvalidAccessException.
+     * Constructs a new JXPathInvalidAccessException.
      * 
      * @param message exception message
      */
@@ -35,7 +35,7 @@ public class JXPathInvalidAccessException extends 
JXPathException {
     }
 
     /**
-     * Create a new JXPathInvalidAccessException.
+     * Constructs a new JXPathInvalidAccessException.
      * 
      * @param message exception message
      * @param ex      precipitating exception
diff --git 
a/src/main/java/org/apache/commons/jxpath/JXPathInvalidSyntaxException.java 
b/src/main/java/org/apache/commons/jxpath/JXPathInvalidSyntaxException.java
index f339742..9393b17 100644
--- a/src/main/java/org/apache/commons/jxpath/JXPathInvalidSyntaxException.java
+++ b/src/main/java/org/apache/commons/jxpath/JXPathInvalidSyntaxException.java
@@ -25,7 +25,7 @@ public class JXPathInvalidSyntaxException extends 
JXPathException {
     private static final long serialVersionUID = 504555366032561816L;
 
     /**
-     * Create a new JXPathInvalidSyntaxException.
+     * Constructs a new JXPathInvalidSyntaxException.
      * 
      * @param message relevant message
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/JXPathNotFoundException.java 
b/src/main/java/org/apache/commons/jxpath/JXPathNotFoundException.java
index 20d4676..0db4140 100644
--- a/src/main/java/org/apache/commons/jxpath/JXPathNotFoundException.java
+++ b/src/main/java/org/apache/commons/jxpath/JXPathNotFoundException.java
@@ -25,7 +25,7 @@ public class JXPathNotFoundException extends JXPathException {
     private static final long serialVersionUID = -8875537628056117241L;
 
     /**
-     * Create a new JXPathNotFoundException.
+     * Constructs a new JXPathNotFoundException.
      * 
      * @param message exception detail
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/JXPathTypeConversionException.java 
b/src/main/java/org/apache/commons/jxpath/JXPathTypeConversionException.java
index b8cb2a2..a21057b 100644
--- a/src/main/java/org/apache/commons/jxpath/JXPathTypeConversionException.java
+++ b/src/main/java/org/apache/commons/jxpath/JXPathTypeConversionException.java
@@ -25,7 +25,7 @@ public class JXPathTypeConversionException extends 
JXPathException {
     private static final long serialVersionUID = -4403564377958943239L;
 
     /**
-     * Create a new JXPathTypeConversionException.
+     * Constructs a new JXPathTypeConversionException.
      * 
      * @param message exception text
      */
@@ -34,7 +34,7 @@ public class JXPathTypeConversionException extends 
JXPathException {
     }
 
     /**
-     * Create a new JXPathTypeConversionException.
+     * Constructs a new JXPathTypeConversionException.
      * 
      * @param message exception text
      * @param ex      underlying cause
diff --git a/src/main/java/org/apache/commons/jxpath/PackageFunctions.java 
b/src/main/java/org/apache/commons/jxpath/PackageFunctions.java
index 089cd42..144d568 100644
--- a/src/main/java/org/apache/commons/jxpath/PackageFunctions.java
+++ b/src/main/java/org/apache/commons/jxpath/PackageFunctions.java
@@ -71,7 +71,7 @@ public class PackageFunctions implements Functions {
     private final String namespace;
 
     /**
-     * Create a new PackageFunctions.
+     * Constructs a new PackageFunctions.
      * 
      * @param classPrefix class prefix
      * @param namespace   namespace String
diff --git a/src/main/java/org/apache/commons/jxpath/XMLDocumentContainer.java 
b/src/main/java/org/apache/commons/jxpath/XMLDocumentContainer.java
index 9d2ad26..e737219 100644
--- a/src/main/java/org/apache/commons/jxpath/XMLDocumentContainer.java
+++ b/src/main/java/org/apache/commons/jxpath/XMLDocumentContainer.java
@@ -38,14 +38,26 @@ import org.apache.commons.jxpath.xml.DocumentContainer;
 public class XMLDocumentContainer implements Container {
 
     private static final long serialVersionUID = 1L;
+
+    /** The delegate document container. */
     private DocumentContainer delegate;
+
+    /** The DOM document. */
     private Object document;
+
+    /**
+     * The document source.
+     */
     private URL xmlURL;
+
+    /**
+     * The transformation source.
+     */
     private Source source;
 
     /**
-     * Create a new XMLDocumentContainer.
-     * 
+     * Constructs a new XMLDocumentContainer.
+     *
      * @param source XML source
      */
     public XMLDocumentContainer(final Source source) {
@@ -53,8 +65,8 @@ public class XMLDocumentContainer implements Container {
     }
 
     /**
-     * Create a new XMLDocumentContainer.
-     * 
+     * Constructs a new XMLDocumentContainer.
+     *
      * @param xmlURL a URL for an XML file. Use 
getClass().getResource(resourceName) to load XML from a resource file.
      */
     public XMLDocumentContainer(final URL xmlURL) {
@@ -64,7 +76,7 @@ public class XMLDocumentContainer implements Container {
 
     /**
      * Reads XML, caches it internally and returns the Document.
-     * 
+     *
      * @return Object value
      */
     @Override
@@ -89,7 +101,7 @@ public class XMLDocumentContainer implements Container {
 
     /**
      * Throws an UnsupportedOperationException
-     * 
+     *
      * @param value to set
      */
     @Override
diff --git 
a/src/main/java/org/apache/commons/jxpath/functions/ConstructorFunction.java 
b/src/main/java/org/apache/commons/jxpath/functions/ConstructorFunction.java
index 9860f08..f22315a 100644
--- a/src/main/java/org/apache/commons/jxpath/functions/ConstructorFunction.java
+++ b/src/main/java/org/apache/commons/jxpath/functions/ConstructorFunction.java
@@ -34,7 +34,7 @@ public class ConstructorFunction implements Function {
     private final Constructor constructor;
 
     /**
-     * Create a new ConstructorFunction.
+     * Constructs a new ConstructorFunction.
      * 
      * @param constructor the constructor to call.
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/functions/MethodFunction.java 
b/src/main/java/org/apache/commons/jxpath/functions/MethodFunction.java
index f07bfe4..ad5e656 100644
--- a/src/main/java/org/apache/commons/jxpath/functions/MethodFunction.java
+++ b/src/main/java/org/apache/commons/jxpath/functions/MethodFunction.java
@@ -36,7 +36,7 @@ public class MethodFunction implements Function {
     private final Method method;
 
     /**
-     * Create a new MethodFunction.
+     * Constructs a new MethodFunction.
      * 
      * @param method implementing Method
      */
diff --git a/src/main/java/org/apache/commons/jxpath/ri/EvalContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/EvalContext.java
index 4877a89..b766dd7 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/EvalContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/EvalContext.java
@@ -54,7 +54,7 @@ public abstract class EvalContext implements 
ExpressionContext, Iterator {
     private Iterator<Pointer> pointerIterator;
 
     /**
-     * Create a new EvalContext.
+     * Constructs a new EvalContext.
      * 
      * @param parentContext parent context
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/JXPathCompiledExpression.java 
b/src/main/java/org/apache/commons/jxpath/ri/JXPathCompiledExpression.java
index cf4a040..f12b8c7 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/JXPathCompiledExpression.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/JXPathCompiledExpression.java
@@ -33,7 +33,7 @@ public class JXPathCompiledExpression implements 
CompiledExpression {
     private final Expression expression;
 
     /**
-     * Create a new JXPathCompiledExpression.
+     * Constructs a new JXPathCompiledExpression.
      * 
      * @param xpath      source
      * @param expression compiled
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/JXPathContextFactoryReferenceImpl.java
 
b/src/main/java/org/apache/commons/jxpath/ri/JXPathContextFactoryReferenceImpl.java
index a52cc1f..e81080a 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/JXPathContextFactoryReferenceImpl.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/JXPathContextFactoryReferenceImpl.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.JXPathContextFactory;
 public class JXPathContextFactoryReferenceImpl extends JXPathContextFactory {
 
     /**
-     * Create a new JXPathContextFactoryReferenceImpl.
+     * Constructs a new JXPathContextFactoryReferenceImpl.
      */
     public JXPathContextFactoryReferenceImpl() {
     }
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/JXPathContextReferenceImpl.java 
b/src/main/java/org/apache/commons/jxpath/ri/JXPathContextReferenceImpl.java
index f3f38c2..101175c 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/JXPathContextReferenceImpl.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/JXPathContextReferenceImpl.java
@@ -176,7 +176,7 @@ public class JXPathContextReferenceImpl extends 
JXPathContext {
     private Pointer contextPointer;
 
     /**
-     * Create a new JXPathContextReferenceImpl.
+     * Constructs a new JXPathContextReferenceImpl.
      * 
      * @param parentContext parent context
      * @param contextBean   Object
@@ -186,7 +186,7 @@ public class JXPathContextReferenceImpl extends 
JXPathContext {
     }
 
     /**
-     * Create a new JXPathContextReferenceImpl.
+     * Constructs a new JXPathContextReferenceImpl.
      * 
      * @param parentContext  parent context
      * @param contextBean    Object
diff --git a/src/main/java/org/apache/commons/jxpath/ri/NamespaceResolver.java 
b/src/main/java/org/apache/commons/jxpath/ri/NamespaceResolver.java
index bacd16b..835b236 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/NamespaceResolver.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/NamespaceResolver.java
@@ -33,7 +33,7 @@ public class NamespaceResolver implements Cloneable, 
Serializable {
 
     /**
      * Find the namespace prefix for the specified namespace URI and 
NodePointer.
-     * 
+     *
      * @param pointer      location
      * @param namespaceURI to check
      * @return prefix if found
@@ -58,26 +58,33 @@ public class NamespaceResolver implements Cloneable, 
Serializable {
         return null;
     }
 
-    /** Parent NamespaceResolver */
+    /** Parent NamespaceResolver. */
     protected final NamespaceResolver parent;
-    /** Namespace map */
+
+    /** Namespace map. */
     protected HashMap namespaceMap = new HashMap();
+
     /** Reverse lookup map */
     protected HashMap reverseMap = new HashMap();
-    /** Pointer */
+
+    /** Node pointer. */
     protected NodePointer pointer;
+
+    /**
+     * Whether this instance is sealed.
+     */
     private boolean sealed;
 
     /**
-     * Create a new NamespaceResolver.
+     * Constructs a new NamespaceResolver.
      */
     public NamespaceResolver() {
         this(null);
     }
 
     /**
-     * Create a new NamespaceResolver.
-     * 
+     * Constructs a new NamespaceResolver.
+     *
      * @param parent NamespaceResolver
      */
     public NamespaceResolver(final NamespaceResolver parent) {
@@ -111,7 +118,7 @@ public class NamespaceResolver implements Cloneable, 
Serializable {
 
     /**
      * Gets the nearest prefix found that matches an externally-registered 
namespace.
-     * 
+     *
      * @param namespaceURI the ns URI to check.
      * @return String prefix if found.
      * @since JXPath 1.3
@@ -123,7 +130,7 @@ public class NamespaceResolver implements Cloneable, 
Serializable {
 
     /**
      * Gets the namespace context pointer.
-     * 
+     *
      * @return Pointer
      */
     public Pointer getNamespaceContextPointer() {
@@ -147,7 +154,7 @@ public class NamespaceResolver implements Cloneable, 
Serializable {
 
     /**
      * Gets the prefix associated with the specifed namespace URI.
-     * 
+     *
      * @param namespaceURI the ns URI to check.
      * @return String prefix
      */
@@ -158,7 +165,7 @@ public class NamespaceResolver implements Cloneable, 
Serializable {
 
     /**
      * Learn whether this NamespaceResolver has been sealed.
-     * 
+     *
      * @return boolean
      */
     public boolean isSealed() {
@@ -191,7 +198,7 @@ public class NamespaceResolver implements Cloneable, 
Serializable {
 
     /**
      * Register a namespace for the expression context.
-     * 
+     *
      * @param pointer the Pointer to set.
      */
     public void setNamespaceContextPointer(final NodePointer pointer) {
diff --git a/src/main/java/org/apache/commons/jxpath/ri/QName.java 
b/src/main/java/org/apache/commons/jxpath/ri/QName.java
index b4ac594..6692cf3 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/QName.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/QName.java
@@ -25,13 +25,25 @@ import java.io.Serializable;
 public class QName implements Serializable {
 
     private static final long serialVersionUID = 7616199282015091496L;
+
+    /**
+     * Prefix.
+     */
     private final String prefix;
+
+    /**
+     * Name.
+     */
     private final String name;
+
+    /**
+     * Qualified name.
+     */
     private final String qualifiedName;
 
     /**
-     * Create a new QName.
-     * 
+     * Constructs a new QName.
+     *
      * @param qualifiedName value
      */
     public QName(final String qualifiedName) {
@@ -42,8 +54,8 @@ public class QName implements Serializable {
     }
 
     /**
-     * Create a new QName.
-     * 
+     * Constructs a new QName.
+     *
      * @param prefix    ns
      * @param localName String
      */
@@ -66,7 +78,7 @@ public class QName implements Serializable {
 
     /**
      * Gets the local name.
-     * 
+     *
      * @return String
      */
     public String getName() {
@@ -75,7 +87,7 @@ public class QName implements Serializable {
 
     /**
      * Gets the prefix of this QName.
-     * 
+     *
      * @return String
      */
     public String getPrefix() {
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/axes/AncestorContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/AncestorContext.java
index 2602714..847fead 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/AncestorContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/AncestorContext.java
@@ -32,7 +32,7 @@ public class AncestorContext extends EvalContext {
     private final boolean includeSelf;
 
     /**
-     * Create a new AncestorContext.
+     * Constructs a new AncestorContext.
      * 
      * @param parentContext represents the previous step on the path
      * @param includeSelf   differentiates between "ancestor::" and 
"ancestor-or-self::" axes
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/axes/AttributeContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/AttributeContext.java
index 0e8b287..dc3807f 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/AttributeContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/AttributeContext.java
@@ -38,7 +38,7 @@ public class AttributeContext extends EvalContext {
     private NodePointer currentNodePointer;
 
     /**
-     * Create a new AttributeContext.
+     * Constructs a new AttributeContext.
      * 
      * @param parentContext represents the previous step on the path
      * @param nodeTest      is the name of the attribute we are looking for
diff --git a/src/main/java/org/apache/commons/jxpath/ri/axes/ChildContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/ChildContext.java
index fb62a42..a0db244 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/ChildContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/ChildContext.java
@@ -34,7 +34,7 @@ public class ChildContext extends EvalContext {
     private NodeIterator iterator;
 
     /**
-     * Create a new ChildContext.
+     * Constructs a new ChildContext.
      * 
      * @param parentContext           parent EvalContext
      * @param nodeTest                NodeTest
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/axes/DescendantContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/DescendantContext.java
index 61a8dd1..6ac632e 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/DescendantContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/DescendantContext.java
@@ -40,7 +40,7 @@ public class DescendantContext extends EvalContext {
     private final boolean includeSelf;
 
     /**
-     * Create a new DescendantContext.
+     * Constructs a new DescendantContext.
      * 
      * @param parentContext parent context
      * @param includeSelf   whether to include this node
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/axes/InitialContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/InitialContext.java
index 8de6d46..6b9043b 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/InitialContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/InitialContext.java
@@ -32,7 +32,7 @@ public class InitialContext extends EvalContext {
     private final NodePointer nodePointer;
 
     /**
-     * Create a new InitialContext.
+     * Constructs a new InitialContext.
      * 
      * @param parentContext parent context
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/axes/NodeSetContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/NodeSetContext.java
index 1fadbde..7058fca 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/NodeSetContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/NodeSetContext.java
@@ -30,7 +30,7 @@ public class NodeSetContext extends EvalContext {
     private final NodeSet nodeSet;
 
     /**
-     * Create a new NodeSetContext.
+     * Constructs a new NodeSetContext.
      * 
      * @param parentContext parent context
      * @param nodeSet       associated NodeSet
diff --git a/src/main/java/org/apache/commons/jxpath/ri/axes/ParentContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/ParentContext.java
index c75866a..84d12f2 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/ParentContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/ParentContext.java
@@ -31,7 +31,7 @@ public class ParentContext extends EvalContext {
     private NodePointer currentNodePointer;
 
     /**
-     * Create a new ParentContext.
+     * Constructs a new ParentContext.
      * 
      * @param parentContext parent context
      * @param nodeTest      test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/axes/PrecedingOrFollowingContext.java
 
b/src/main/java/org/apache/commons/jxpath/ri/axes/PrecedingOrFollowingContext.java
index 0d74658..6fb9c94 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/axes/PrecedingOrFollowingContext.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/axes/PrecedingOrFollowingContext.java
@@ -37,7 +37,7 @@ public class PrecedingOrFollowingContext extends EvalContext {
     private final boolean reverse;
 
     /**
-     * Create a new PrecedingOrFollowingContext.
+     * Constructs a new PrecedingOrFollowingContext.
      * 
      * @param parentContext parent context
      * @param nodeTest      test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/axes/PredicateContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/PredicateContext.java
index bf2eb70..38648cd 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/PredicateContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/PredicateContext.java
@@ -38,7 +38,7 @@ public class PredicateContext extends EvalContext {
     private PropertyPointer dynamicPropertyPointer;
 
     /**
-     * Create a new PredicateContext.
+     * Constructs a new PredicateContext.
      * 
      * @param parentContext parent context
      * @param expression    compiled Expression
diff --git a/src/main/java/org/apache/commons/jxpath/ri/axes/RootContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/RootContext.java
index 84ea6b5..b82fc6d 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/RootContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/RootContext.java
@@ -42,7 +42,7 @@ public class RootContext extends EvalContext {
     private int availableRegister;
 
     /**
-     * Create a new RootContext.
+     * Constructs a new RootContext.
      * 
      * @param jxpathContext context
      * @param pointer       pointer
diff --git a/src/main/java/org/apache/commons/jxpath/ri/axes/SelfContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/SelfContext.java
index 6b81d6c..b75e7bf 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/SelfContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/SelfContext.java
@@ -32,7 +32,7 @@ public class SelfContext extends EvalContext {
     private NodePointer nodePointer;
 
     /**
-     * Create a new SelfContext.
+     * Constructs a new SelfContext.
      * 
      * @param parentContext EvalContext
      * @param nodeTest      guard
diff --git a/src/main/java/org/apache/commons/jxpath/ri/axes/UnionContext.java 
b/src/main/java/org/apache/commons/jxpath/ri/axes/UnionContext.java
index 54ad728..37c311e 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/axes/UnionContext.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/axes/UnionContext.java
@@ -33,7 +33,7 @@ public class UnionContext extends NodeSetContext {
     private boolean prepared;
 
     /**
-     * Create a new UnionContext.
+     * Constructs a new UnionContext.
      * 
      * @param parentContext parent context
      * @param contexts      child contexts
diff --git a/src/main/java/org/apache/commons/jxpath/ri/compiler/Constant.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/Constant.java
index cca4d71..24a4b4f 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/Constant.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/Constant.java
@@ -28,7 +28,7 @@ public class Constant extends Expression {
     private final Object value;
 
     /**
-     * Create a new Constant.
+     * Constructs a new Constant.
      * 
      * @param number constant
      */
@@ -37,7 +37,7 @@ public class Constant extends Expression {
     }
 
     /**
-     * Create a new Constant.
+     * Constructs a new Constant.
      * 
      * @param string constant
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreFunction.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreFunction.java
index 4836748..b892710 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreFunction.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreFunction.java
@@ -43,7 +43,7 @@ public class CoreFunction extends Operation {
     private final int functionCode;
 
     /**
-     * Create a new CoreFunction.
+     * Constructs a new CoreFunction.
      * 
      * @param functionCode int function code
      * @param args         argument Expressions
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperation.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperation.java
index d2994f6..b257b91 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperation.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperation.java
@@ -42,7 +42,7 @@ public abstract class CoreOperation extends Operation {
     protected static final int UNION_PRECEDENCE = 7;
 
     /**
-     * Create a new CoreOperation.
+     * Constructs a new CoreOperation.
      * 
      * @param args Expression[]
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAdd.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAdd.java
index d6d6cf4..31cde4e 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAdd.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAdd.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationAdd extends CoreOperation {
 
     /**
-     * Create a new CoreOperationAdd.
+     * Constructs a new CoreOperationAdd.
      * 
      * @param args Expression arguments to add together.
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAnd.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAnd.java
index 5881fb7..d00fffd 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAnd.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationAnd.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationAnd extends CoreOperation {
 
     /**
-     * Create a new CoreOperationAnd.
+     * Constructs a new CoreOperationAnd.
      * 
      * @param args to combine
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationCompare.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationCompare.java
index d31d531..fed9e2c 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationCompare.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationCompare.java
@@ -35,7 +35,7 @@ public abstract class CoreOperationCompare extends 
CoreOperation {
     private final boolean invert;
 
     /**
-     * Create a new CoreOperationCompare.
+     * Constructs a new CoreOperationCompare.
      * 
      * @param arg1 left operand
      * @param arg2 right operand
@@ -45,7 +45,7 @@ public abstract class CoreOperationCompare extends 
CoreOperation {
     }
 
     /**
-     * Create a new CoreOperationCompare.
+     * Constructs a new CoreOperationCompare.
      * 
      * @param arg1   left operand
      * @param arg2   right operand
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationDivide.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationDivide.java
index 8019ba2..b80adcf 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationDivide.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationDivide.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationDivide extends CoreOperation {
 
     /**
-     * Create a new CoreOperationDivide.
+     * Constructs a new CoreOperationDivide.
      * 
      * @param arg1 dividend
      * @param arg2 divisor
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationEqual.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationEqual.java
index 2dbb56d..83f0d0c 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationEqual.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationEqual.java
@@ -23,7 +23,7 @@ package org.apache.commons.jxpath.ri.compiler;
 public class CoreOperationEqual extends CoreOperationCompare {
 
     /**
-     * Create a new CoreOperationEqual
+     * Constructs a new CoreOperationEqual
      * 
      * @param arg1 first comparison Expression
      * @param arg2 second comparison Expression
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThan.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThan.java
index 4d19258..bc652d2 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThan.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThan.java
@@ -23,7 +23,7 @@ package org.apache.commons.jxpath.ri.compiler;
 public class CoreOperationGreaterThan extends 
CoreOperationRelationalExpression {
 
     /**
-     * Create a new CoreOperationGreaterThan.
+     * Constructs a new CoreOperationGreaterThan.
      * 
      * @param arg1 left operand
      * @param arg2 right operand
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThanOrEqual.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThanOrEqual.java
index de4d2ab..837ba42 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThanOrEqual.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationGreaterThanOrEqual.java
@@ -23,7 +23,7 @@ package org.apache.commons.jxpath.ri.compiler;
 public class CoreOperationGreaterThanOrEqual extends 
CoreOperationRelationalExpression {
 
     /**
-     * Create a new CoreOperationGreaterThanOrEqual.
+     * Constructs a new CoreOperationGreaterThanOrEqual.
      * 
      * @param arg1 operand 1
      * @param arg2 operand 2
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThan.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThan.java
index 71c1526..b8fa880 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThan.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThan.java
@@ -23,7 +23,7 @@ package org.apache.commons.jxpath.ri.compiler;
 public class CoreOperationLessThan extends CoreOperationRelationalExpression {
 
     /**
-     * Create a new CoreOperationLessThan.
+     * Constructs a new CoreOperationLessThan.
      * 
      * @param arg1 left Expression
      * @param arg2 right Expression
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThanOrEqual.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThanOrEqual.java
index 4c31d6a..700fba6 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThanOrEqual.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationLessThanOrEqual.java
@@ -23,7 +23,7 @@ package org.apache.commons.jxpath.ri.compiler;
 public class CoreOperationLessThanOrEqual extends 
CoreOperationRelationalExpression {
 
     /**
-     * Create a new CoreOperationLessThanOrEqual.
+     * Constructs a new CoreOperationLessThanOrEqual.
      * 
      * @param arg1 left Expression
      * @param arg2 right Expression
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMod.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMod.java
index ef20177..eaac9f8 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMod.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMod.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationMod extends CoreOperation {
 
     /**
-     * Create a new CoreOperationMod.
+     * Constructs a new CoreOperationMod.
      * 
      * @param arg1 dividend
      * @param arg2 divisor
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMultiply.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMultiply.java
index b17308d..5a0b04f 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMultiply.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationMultiply.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationMultiply extends CoreOperation {
 
     /**
-     * Create a new CoreOperationMultiply.
+     * Constructs a new CoreOperationMultiply.
      * 
      * @param arg1 factor 1
      * @param arg2 factor 2
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNegate.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNegate.java
index 162b1f8..2dcd932 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNegate.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNegate.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationNegate extends CoreOperation {
 
     /**
-     * Create a new CoreOperationNegate.
+     * Constructs a new CoreOperationNegate.
      * 
      * @param arg the Expression to negate
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNotEqual.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNotEqual.java
index 7bc3426..c2be065 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNotEqual.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationNotEqual.java
@@ -23,7 +23,7 @@ package org.apache.commons.jxpath.ri.compiler;
 public class CoreOperationNotEqual extends CoreOperationCompare {
 
     /**
-     * Create a new CoreOperationNotEqual.
+     * Constructs a new CoreOperationNotEqual.
      * 
      * @param arg1 left operand
      * @param arg2 right operand
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationOr.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationOr.java
index 07f7df3..e8ee6cd 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationOr.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationOr.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationOr extends CoreOperation {
 
     /**
-     * Create a new CoreOperationOr.
+     * Constructs a new CoreOperationOr.
      * 
      * @param args or'd Expression components
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationRelationalExpression.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationRelationalExpression.java
index 23917ef..d29073a 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationRelationalExpression.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationRelationalExpression.java
@@ -34,7 +34,7 @@ import org.apache.commons.jxpath.ri.axes.SelfContext;
 public abstract class CoreOperationRelationalExpression extends CoreOperation {
 
     /**
-     * Create a new CoreOperationRelationalExpression.
+     * Constructs a new CoreOperationRelationalExpression.
      * 
      * @param args arguments
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationSubtract.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationSubtract.java
index 127c191..109555c 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationSubtract.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationSubtract.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.InfoSetUtil;
 public class CoreOperationSubtract extends CoreOperation {
 
     /**
-     * Create a new CoreOperationSubtract.
+     * Constructs a new CoreOperationSubtract.
      * 
      * @param arg1 minuend
      * @param arg2 subtrahend
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationUnion.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationUnion.java
index 579740b..94def81 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationUnion.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/CoreOperationUnion.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.ri.axes.UnionContext;
 public class CoreOperationUnion extends CoreOperation {
 
     /**
-     * Create a new CoreOperationUnion.
+     * Constructs a new CoreOperationUnion.
      * 
      * @param args Expression[]
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/Expression.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/Expression.java
index eb67917..b9dd66f 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/Expression.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/Expression.java
@@ -45,7 +45,7 @@ public abstract class Expression {
 
         // to what method does the following comment refer?
         /**
-         * Create a new PointerIterator
+         * Constructs a new PointerIterator
          * 
          * @param it     underlying Iterator
          * @param qname  name
@@ -87,7 +87,7 @@ public abstract class Expression {
         private final Iterator iterator;
 
         /**
-         * Create a new ValueIterator.
+         * Constructs a new ValueIterator.
          * 
          * @param it underlying Iterator, may contain pointers
          */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/ExpressionPath.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/ExpressionPath.java
index 90598a0..2a05acc 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/ExpressionPath.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/ExpressionPath.java
@@ -37,7 +37,7 @@ public class ExpressionPath extends Path {
     private boolean basic;
 
     /**
-     * Create a new ExpressionPath.
+     * Constructs a new ExpressionPath.
      * 
      * @param expression Expression
      * @param predicates to execute
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunction.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunction.java
index bb73b08..ee67c47 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunction.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunction.java
@@ -34,7 +34,7 @@ public class ExtensionFunction extends Operation {
     private final QName functionName;
 
     /**
-     * Create a new ExtensionFunction.
+     * Constructs a new ExtensionFunction.
      * 
      * @param functionName name of the function
      * @param args         Expression[] of function args
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/LocationPath.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/LocationPath.java
index 58627b4..59c72f7 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/LocationPath.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/LocationPath.java
@@ -28,7 +28,7 @@ public class LocationPath extends Path {
     private final boolean absolute;
 
     /**
-     * Create a new LocationPath.
+     * Constructs a new LocationPath.
      * 
      * @param absolute whether this is an absolute path
      * @param steps    to evaluate
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/NameAttributeTest.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/NameAttributeTest.java
index c18a1ff..9615948 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/NameAttributeTest.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/NameAttributeTest.java
@@ -23,7 +23,7 @@ package org.apache.commons.jxpath.ri.compiler;
 public class NameAttributeTest extends CoreOperationEqual {
 
     /**
-     * Create a new NameAttributeTest.
+     * Constructs a new NameAttributeTest.
      * 
      * @param namePath  Expression
      * @param nameValue Expression
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeNameTest.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeNameTest.java
index 37e2dff..6ffde73 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeNameTest.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeNameTest.java
@@ -28,7 +28,7 @@ public class NodeNameTest extends NodeTest {
     private String namespaceURI;
 
     /**
-     * Create a new NodeNameTest.
+     * Constructs a new NodeNameTest.
      * 
      * @param qname name to match
      */
@@ -37,7 +37,7 @@ public class NodeNameTest extends NodeTest {
     }
 
     /**
-     * Create a new NodeNameTest.
+     * Constructs a new NodeNameTest.
      * 
      * @param qname        name to match
      * @param namespaceURI uri to match
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeTypeTest.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeTypeTest.java
index 6951522..6a45064 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeTypeTest.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/NodeTypeTest.java
@@ -48,7 +48,7 @@ public class NodeTypeTest extends NodeTest {
     private final int nodeType;
 
     /**
-     * Create a new NodeTypeTest.
+     * Constructs a new NodeTypeTest.
      * 
      * @param nodeType to match
      */
diff --git a/src/main/java/org/apache/commons/jxpath/ri/compiler/Operation.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/Operation.java
index 98453ff..f759ab6 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/Operation.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/Operation.java
@@ -26,7 +26,7 @@ public abstract class Operation extends Expression {
     protected Expression[] args;
 
     /**
-     * Create a new Operation.
+     * Constructs a new Operation.
      * 
      * @param args Expression[]
      */
diff --git a/src/main/java/org/apache/commons/jxpath/ri/compiler/Path.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/Path.java
index e0a6601..1784dc4 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/Path.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/Path.java
@@ -45,7 +45,7 @@ public abstract class Path extends Expression {
     private boolean basic;
 
     /**
-     * Create a new Path.
+     * Constructs a new Path.
      * 
      * @param steps that compose the Path
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/ProcessingInstructionTest.java
 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/ProcessingInstructionTest.java
index 57f7ed2..43ab65a 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/ProcessingInstructionTest.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/ProcessingInstructionTest.java
@@ -25,7 +25,7 @@ public class ProcessingInstructionTest extends NodeTest {
     private final String target;
 
     /**
-     * Create a new ProcessingInstructionTest.
+     * Constructs a new ProcessingInstructionTest.
      * 
      * @param target string
      */
diff --git a/src/main/java/org/apache/commons/jxpath/ri/compiler/Step.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/Step.java
index 211315c..c4e0982 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/Step.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/Step.java
@@ -70,7 +70,7 @@ public class Step {
     private final Expression[] predicates;
 
     /**
-     * Create a new Step.
+     * Constructs a new Step.
      * 
      * @param axis       axis code
      * @param nodeTest   step test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/compiler/VariableReference.java 
b/src/main/java/org/apache/commons/jxpath/ri/compiler/VariableReference.java
index c83af14..6824ed7 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/compiler/VariableReference.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/compiler/VariableReference.java
@@ -28,7 +28,7 @@ public class VariableReference extends Expression {
     private final QName varName;
 
     /**
-     * Create a new VariableReference.
+     * Constructs a new VariableReference.
      * 
      * @param varName variable name
      */
diff --git a/src/main/java/org/apache/commons/jxpath/ri/model/NodePointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/NodePointer.java
index e554b78..3ec3d30 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/NodePointer.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/NodePointer.java
@@ -51,7 +51,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Allocates an new child NodePointer by iterating through all installed 
NodePointerFactories until it finds one that can create a pointer.
-     * 
+     *
      * @param parent pointer
      * @param name   QName
      * @param bean   Object
@@ -70,7 +70,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Allocates an entirely new NodePointer by iterating through all 
installed NodePointerFactories until it finds one that can create a pointer.
-     * 
+     *
      * @param name   QName
      * @param bean   Object
      * @param locale Locale
@@ -94,7 +94,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Print deep
-     * 
+     *
      * @param pointer to print
      * @param indent  indentation level
      */
@@ -115,7 +115,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Verify the structure of a given NodePointer.
-     * 
+     *
      * @param nodePointer to check
      * @return nodePointer
      * @throws JXPathNotFoundException Thrown when there is no value at the 
NodePointer.
@@ -136,20 +136,38 @@ public abstract class NodePointer implements Pointer {
         return nodePointer;
     }
 
-    /** Index for this NodePointer */
+    /** Index for this NodePointer. */
     protected int index = WHOLE_COLLECTION;
+
+    /**
+     * Whether this is an attribute.
+     */
     private boolean attribute;
+
+    /**
+     * Namespace resolver.
+     */
     private NamespaceResolver namespaceResolver;
+
+    /**
+     * Exception handler for {@link #handle(Throwable, NodePointer)}.
+     */
     private ExceptionHandler exceptionHandler;
+
+    /**
+     * Root node.
+     */
     private transient Object rootNode;
+
     /** Parent pointer */
     protected NodePointer parent;
+
     /** Locale */
     protected Locale locale;
 
     /**
-     * Create a new NodePointer.
-     * 
+     * Constructs a new NodePointer.
+     *
      * @param parent Pointer
      */
     protected NodePointer(final NodePointer parent) {
@@ -157,8 +175,8 @@ public abstract class NodePointer implements Pointer {
     }
 
     /**
-     * Create a new NodePointer.
-     * 
+     * Constructs a new NodePointer.
+     *
      * @param parent Pointer
      * @param locale Locale
      */
@@ -169,7 +187,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns an XPath that maps to this Pointer.
-     * 
+     *
      * @return String xpath expression
      */
     @Override
@@ -199,7 +217,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Returns a NodeIterator that iterates over all attributes of the current 
node matching the supplied node name (could have a wildcard). May return null if
      * the object does not support the attributes.
-     * 
+     *
      * @param qname the attribute name to test
      * @return NodeIterator
      */
@@ -210,7 +228,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns a NodeIterator that iterates over all children or all children 
that match the given NodeTest, starting with the specified one.
-     * 
+     *
      * @param test      NodeTest to filter children
      * @param reverse   specified iteration direction
      * @param startWith the NodePointer to start with
@@ -223,7 +241,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Clone this NodePointer.
-     * 
+     *
      * @return cloned NodePointer
      */
     @Override
@@ -243,7 +261,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Compares two child NodePointers and returns a positive number, zero or 
a positive number according to the order of the pointers.
-     * 
+     *
      * @param pointer1 first pointer to be compared
      * @param pointer2 second pointer to be compared
      * @return int per Java comparison conventions
@@ -252,7 +270,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Compare node pointers.
-     * 
+     *
      * @param p1     pointer 1
      * @param depth1 depth 1
      * @param p2     pointer 2
@@ -316,7 +334,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Called to create a non-existing attribute
-     * 
+     *
      * @param context the owning JXPathCOntext
      * @param name    the QName at which an attribute should be created
      * @return created NodePointer
@@ -328,7 +346,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Called by a child pointer when it needs to create a parent object for a 
non-existent collection element. It may have to expand the collection, then
      * create an element object and return a new pointer describing the newly 
created element.
-     * 
+     *
      * @param context the owning JXPathCOntext
      * @param name    the QName at which a child should be created
      * @param index   child index.
@@ -342,7 +360,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Called by a child pointer if that child needs to assign the value 
supplied in the createPath(context, value) call to a non-existent node. This 
method may
      * have to expand the collection in order to assign the element.
-     * 
+     *
      * @param context the owning JXPathCOntext
      * @param name    the QName at which a child should be created
      * @param index   child index.
@@ -357,7 +375,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Called by a child pointer when it needs to create a parent object. Must 
create an object described by this pointer and return a new pointer that 
properly
      * describes the new object.
-     * 
+     *
      * @param context the owning JXPathContext
      * @return created NodePointer
      */
@@ -367,7 +385,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Called directly by JXPathContext. Must create path and set value.
-     * 
+     *
      * @param context the owning JXPathContext
      * @param value   the new value to set
      * @return created NodePointer
@@ -379,7 +397,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Return a string escaping single and double quotes.
-     * 
+     *
      * @param string string to treat
      * @return string with any necessary changes made.
      */
@@ -414,7 +432,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Gets the AbstractFactory associated with the specified JXPathContext.
-     * 
+     *
      * @param context JXPathContext
      * @return AbstractFactory
      */
@@ -429,14 +447,14 @@ public abstract class NodePointer implements Pointer {
     /**
      * Returns the value represented by the pointer before indexing. So, if 
the node represents an element of a collection, this method returns the 
collection
      * itself.
-     * 
+     *
      * @return Object value
      */
     public abstract Object getBaseValue();
 
     /**
      * Gets the default ns uri
-     * 
+     *
      * @return String uri
      */
     protected String getDefaultNamespaceURI() {
@@ -445,14 +463,14 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns the object the pointer points to; does not convert it to a 
"canonical" type.
-     * 
+     *
      * @return Object node
      */
     public abstract Object getImmediateNode();
 
     /**
      * Gets the immediate parent pointer.
-     * 
+     *
      * @return NodePointer
      */
     public NodePointer getImmediateParentPointer() {
@@ -472,7 +490,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * If the pointer represents a collection, the index identifies an element 
of that collection. The default value of {@code index} is
      * {@code WHOLE_COLLECTION}, which just means that the pointer is not 
indexed at all. Note: the index on NodePointer starts with 0, not 1.
-     * 
+     *
      * @return the index.
      */
     public int getIndex() {
@@ -481,14 +499,14 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * If the pointer represents a collection (or collection element), returns 
the length of the collection. Otherwise returns 1 (even if the value is null).
-     * 
+     *
      * @return the length.
      */
     public abstract int getLength();
 
     /**
      * If the Pointer has a parent, returns the parent's locale; otherwise 
returns the locale specified when this Pointer was created.
-     * 
+     *
      * @return Locale for this NodePointer
      */
     public Locale getLocale() {
@@ -500,14 +518,14 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Gets the name of this node. Can be null.
-     * 
+     *
      * @return QName The name of this node. Can be null.
      */
     public abstract QName getName();
 
     /**
      * Gets the NamespaceResolver associated with this NodePointer.
-     * 
+     *
      * @return NamespaceResolver
      */
     public NamespaceResolver getNamespaceResolver() {
@@ -519,7 +537,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns the namespace URI associated with this Pointer.
-     * 
+     *
      * @return String uri
      */
     public String getNamespaceURI() {
@@ -528,7 +546,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Decodes a namespace prefix to the corresponding URI.
-     * 
+     *
      * @param prefix prefix to decode
      * @return String uri
      */
@@ -538,7 +556,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns the object the pointer points to; does not convert it to a 
"canonical" type. Opens containers, properties etc and returns the ultimate 
contents.
-     * 
+     *
      * @return Object node
      */
     @Override
@@ -548,7 +566,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Find a NodeSet by key/value.
-     * 
+     *
      * @param context owning JXPathContext
      * @param key     key to search for
      * @param value   value to match
@@ -560,7 +578,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns the object the pointer points to; does not convert it to a 
"canonical" type.
-     * 
+     *
      * @return Object node value
      * @deprecated 1.1 Please use getNode()
      */
@@ -571,7 +589,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Gets the parent pointer.
-     * 
+     *
      * @return NodePointer
      */
     public NodePointer getParent() {
@@ -584,7 +602,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Locates a node by ID.
-     * 
+     *
      * @param context JXPathContext owning context
      * @param id      String id
      * @return Pointer found
@@ -595,7 +613,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Locates a node by key and value.
-     * 
+     *
      * @param context owning JXPathContext
      * @param key     key to search for
      * @param value   value to match
@@ -607,7 +625,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Gets the root node.
-     * 
+     *
      * @return Object value of this pointer's root (top parent).
      */
     @Override
@@ -620,7 +638,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * By default, returns {@code getNode()}, can be overridden to return a 
"canonical" value, like for instance a DOM element should return its string 
value.
-     * 
+     *
      * @return Object value
      */
     @Override
@@ -640,7 +658,7 @@ public abstract class NodePointer implements Pointer {
      * not {@code this}, invokes {@code getValuePointer()} recursively. The 
idea here is to open all nested containers. Let's say we have a container 
within a
      * container within a container. The {@code getValuePointer()} method 
should then open all those containers and return the pointer to the ultimate 
contents.
      * It does so with the above recursion.
-     * 
+     *
      * @return NodePointer
      */
     public NodePointer getValuePointer() {
@@ -651,7 +669,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Handle a Throwable using an installed ExceptionHandler, if available. 
Public to facilitate calling for RI support; not truly intended for public
      * consumption.
-     * 
+     *
      * @param t to handle
      */
     public void handle(final Throwable t) {
@@ -661,7 +679,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Handle a Throwable using an installed ExceptionHandler, if available. 
Public to facilitate calling for RI support; not truly intended for public
      * consumption.
-     * 
+     *
      * @param t          to handle
      * @param originator context
      */
@@ -680,7 +698,7 @@ public abstract class NodePointer implements Pointer {
      * For instance consider the pointer "/address/street". If both 
<em>address</em> and <em>street</em> are not null, the pointer is actual. If
      * <em>address</em> is not null, but <em>street</em> is null, the pointer 
is still actual. If <em>address</em> is null, the pointer is not actual. (In
      * JavaBeans) if <em>address</em> is not a property of the root bean, a 
Pointer for this path cannot be obtained at all - actual or otherwise.
-     * 
+     *
      * @return boolean
      */
     public boolean isActual() {
@@ -689,7 +707,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns true if the pointer represents the "attribute::" axis.
-     * 
+     *
      * @return boolean
      */
     public boolean isAttribute() {
@@ -698,14 +716,14 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns {@code true} if the value of the pointer is an array or a 
Collection.
-     * 
+     *
      * @return boolean
      */
     public abstract boolean isCollection();
 
     /**
      * If true, this node is auxiliary and can only be used as an intermediate 
in the chain of pointers.
-     * 
+     *
      * @return boolean
      */
     public boolean isContainer() {
@@ -714,7 +732,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns true if the supplied prefix represents the default namespace in 
the context of the current node.
-     * 
+     *
      * @param prefix the prefix to check
      * @return {@code true} if prefix is default
      */
@@ -728,7 +746,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Check whether our locale matches the specified language.
-     * 
+     *
      * @param lang String language to check
      * @return true if the selected locale name starts with the specified 
prefix <em>lang</em>, case-insensitive.
      */
@@ -740,14 +758,14 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * If true, this node does not have children
-     * 
+     *
      * @return boolean
      */
     public abstract boolean isLeaf();
 
     /**
      * Learn whether this pointer is considered to be a node.
-     * 
+     *
      * @return boolean
      * @deprecated Please use !isContainer()
      */
@@ -758,7 +776,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Returns true if this Pointer has no parent.
-     * 
+     *
      * @return boolean
      */
     public boolean isRoot() {
@@ -768,7 +786,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Returns a NodeIterator that iterates over all namespaces of the value 
currently pointed at. May return null if the object does not support the
      * namespaces.
-     * 
+     *
      * @return NodeIterator
      */
     public NodeIterator namespaceIterator() {
@@ -778,7 +796,7 @@ public abstract class NodePointer implements Pointer {
     /**
      * Returns a NodePointer for the specified namespace. Will return null if 
namespaces are not supported. Will return UNKNOWN_NAMESPACE if there is no such
      * namespace.
-     * 
+     *
      * @param namespace incoming namespace
      * @return NodePointer for {@code namespace}
      */
@@ -804,7 +822,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Sets to true if the pointer represents the "attribute::" axis.
-     * 
+     *
      * @param attribute boolean
      */
     public void setAttribute(final boolean attribute) {
@@ -813,7 +831,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Sets the exceptionHandler of this NodePointer.
-     * 
+     *
      * @param exceptionHandler the ExceptionHandler to set
      */
     public void setExceptionHandler(final ExceptionHandler exceptionHandler) {
@@ -822,7 +840,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Sets the index of this NodePointer.
-     * 
+     *
      * @param index int
      */
     public void setIndex(final int index) {
@@ -831,7 +849,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Sets the NamespaceResolver for this NodePointer.
-     * 
+     *
      * @param namespaceResolver NamespaceResolver
      */
     public void setNamespaceResolver(final NamespaceResolver 
namespaceResolver) {
@@ -840,7 +858,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Converts the value to the required type and changes the corresponding 
object to that value.
-     * 
+     *
      * @param value the value to set
      */
     @Override
@@ -848,7 +866,7 @@ public abstract class NodePointer implements Pointer {
 
     /**
      * Checks if this Pointer matches the supplied NodeTest.
-     * 
+     *
      * @param test the NodeTest to execute
      * @return true if a match
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java
index 8fca584..d65bdcf 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointer.java
@@ -35,14 +35,30 @@ import org.apache.commons.jxpath.util.ValueUtils;
 public class VariablePointer extends NodePointer {
 
     private static final long serialVersionUID = -454731297397189293L;
+
+    /**
+     * Variables.
+     */
     private Variables variables;
+
+    /**
+     * Qualified name.
+     */
     private final QName name;
+
+    /**
+     * Value pointer.
+     */
     private NodePointer valuePointer;
+
+    /**
+     * Implements {@link NodePointer#isActual()}.
+     */
     private boolean actual;
 
     /**
-     * Create a new (non-actual) VariablePointer.
-     * 
+     * Constructs a new (non-actual) VariablePointer.
+     *
      * @param name variable name
      */
     public VariablePointer(final QName name) {
@@ -52,8 +68,8 @@ public class VariablePointer extends NodePointer {
     }
 
     /**
-     * Create a new VariablePointer.
-     * 
+     * Constructs a new VariablePointer.
+     *
      * @param variables Variables instance
      * @param name      variable name
      */
@@ -121,7 +137,7 @@ public class VariablePointer extends NodePointer {
 
     /**
      * Create a collection.
-     * 
+     *
      * @param context JXPathContext
      * @param index   collection index
      * @return Object
@@ -182,7 +198,7 @@ public class VariablePointer extends NodePointer {
 
     /**
      * Assimilate the Variables instance associated with the specified context.
-     * 
+     *
      * @param context JXPathContext to search
      */
     protected void findVariables(final JXPathContext context) {
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointerFactory.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointerFactory.java
index 95e706c..4c8f111 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointerFactory.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/VariablePointerFactory.java
@@ -38,7 +38,7 @@ public class VariablePointerFactory implements 
NodePointerFactory {
         private final JXPathContext context;
 
         /**
-         * Create a new VariableContextWrapper.
+         * Constructs a new VariableContextWrapper.
          * 
          * @param context to wrap
          */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanAttributeIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanAttributeIterator.java
index 8d7171c..8896c5f 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanAttributeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanAttributeIterator.java
@@ -30,7 +30,7 @@ public class BeanAttributeIterator extends PropertyIterator {
     private final boolean includeXmlLang;
 
     /**
-     * Create a new BeanAttributeIterator.
+     * Constructs a new BeanAttributeIterator.
      * 
      * @param parent parent pointer
      * @param name   name of this bean
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPointer.java
index a9e7039..bbf1170 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPointer.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPointer.java
@@ -31,12 +31,24 @@ import org.apache.commons.jxpath.ri.model.NodePointer;
 public class BeanPointer extends PropertyOwnerPointer {
 
     private static final long serialVersionUID = -8227317938284982440L;
+    
+    /**
+     * Qualified name.
+     */
     private final QName name;
+    
+    /**
+     * Bean.
+     */
     private final Object bean;
+    
+    /**
+     * Bean info.
+     */
     private final JXPathBeanInfo beanInfo;
 
     /**
-     * Create a new BeanPointer.
+     * BeanPointer a new BeanPointer.
      * 
      * @param parent   pointer
      * @param name     is the name given to the first node
@@ -51,7 +63,7 @@ public class BeanPointer extends PropertyOwnerPointer {
     }
 
     /**
-     * Create a new BeanPointer.
+     * Constructs a new BeanPointer.
      * 
      * @param name     is the name given to the first node
      * @param bean     pointed
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPropertyPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPropertyPointer.java
index 33d076a..2425e68 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPropertyPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/BeanPropertyPointer.java
@@ -42,7 +42,7 @@ public class BeanPropertyPointer extends PropertyPointer {
     private transient PropertyDescriptor propertyDescriptor;
 
     /**
-     * Create a new BeanPropertyPointer.
+     * Constructs a new BeanPropertyPointer.
      * 
      * @param parent   parent pointer
      * @param beanInfo describes the target property/ies.
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionAttributeNodeIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionAttributeNodeIterator.java
index ae6b145..5c29edb 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionAttributeNodeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionAttributeNodeIterator.java
@@ -29,7 +29,7 @@ public class CollectionAttributeNodeIterator extends 
CollectionNodeIterator {
     private final QName name;
 
     /**
-     * Create a new CollectionAttributeNodeIterator.
+     * Constructs a new CollectionAttributeNodeIterator.
      * 
      * @param pointer collection pointer
      * @param name    attribute name
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionChildNodeIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionChildNodeIterator.java
index 7fa9cf3..d2cde7f 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionChildNodeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionChildNodeIterator.java
@@ -29,7 +29,7 @@ public class CollectionChildNodeIterator extends 
CollectionNodeIterator {
     private final NodeTest test;
 
     /**
-     * Create a new CollectionChildNodeIterator.
+     * Constructs a new CollectionChildNodeIterator.
      * 
      * @param pointer   CollectionPointer
      * @param test      child test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionNodeIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionNodeIterator.java
index 4ff6065..b27d44f 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionNodeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionNodeIterator.java
@@ -36,7 +36,7 @@ public abstract class CollectionNodeIterator implements 
NodeIterator {
     private List<NodePointer> collection;
 
     /**
-     * Create a new CollectionNodeIterator.
+     * Constructs a new CollectionNodeIterator.
      * 
      * @param pointer   collection pointer
      * @param reverse   iteration order
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionPointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionPointer.java
index 606facf..52e1c53 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/CollectionPointer.java
@@ -40,7 +40,7 @@ public class CollectionPointer extends NodePointer {
     private NodePointer valuePointer;
 
     /**
-     * Create a new CollectionPointer.
+     * Constructs a new CollectionPointer.
      * 
      * @param parent     parent NodePointer
      * @param collection value
@@ -51,7 +51,7 @@ public class CollectionPointer extends NodePointer {
     }
 
     /**
-     * Create a new CollectionPointer.
+     * Constructs a new CollectionPointer.
      * 
      * @param collection value
      * @param locale     Locale
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/LangAttributePointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/LangAttributePointer.java
index 8cf339e..50475a1 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/LangAttributePointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/LangAttributePointer.java
@@ -29,7 +29,7 @@ public class LangAttributePointer extends NodePointer {
     private static final long serialVersionUID = -8665319197100034134L;
 
     /**
-     * Create a new LangAttributePointer.
+     * Constructs a new LangAttributePointer.
      * 
      * @param parent parent pointer.
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullElementPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullElementPointer.java
index ab978c4..5bb4d41 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullElementPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullElementPointer.java
@@ -31,7 +31,7 @@ public class NullElementPointer extends CollectionPointer {
     private static final long serialVersionUID = 8714236818791036721L;
 
     /**
-     * Create a new NullElementPointer.
+     * Constructs a new NullElementPointer.
      * 
      * @param parent parent pointer
      * @param index  int
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPointer.java
index 7ea50ae..34b2783 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPointer.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPointer.java
@@ -33,7 +33,7 @@ public class NullPointer extends PropertyOwnerPointer {
     private String id;
 
     /**
-     * Create a new NullPointer.
+     * Constructs a new NullPointer.
      * 
      * @param locale Locale
      * @param id     String
@@ -55,7 +55,7 @@ public class NullPointer extends PropertyOwnerPointer {
     }
 
     /**
-     * Create a new NullPointer.
+     * Constructs a new NullPointer.
      * 
      * @param name   node name
      * @param locale Locale
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPropertyPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPropertyPointer.java
index 5d84201..692c8cb 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPropertyPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/NullPropertyPointer.java
@@ -34,7 +34,7 @@ public class NullPropertyPointer extends PropertyPointer {
     private boolean byNameAttribute;
 
     /**
-     * Create a new NullPropertyPointer.
+     * Constructs a new NullPropertyPointer.
      * 
      * @param parent pointer
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyIterator.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyIterator.java
index 1c08680..e6df16f 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyIterator.java
@@ -38,7 +38,7 @@ public class PropertyIterator implements NodeIterator {
     private boolean includeStart;
 
     /**
-     * Create a new PropertyIterator.
+     * Constructs a new PropertyIterator.
      * 
      * @param pointer   owning pointer
      * @param name      property name
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyOwnerPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyOwnerPointer.java
index 4d6bfbd..f0e0c15 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyOwnerPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/beans/PropertyOwnerPointer.java
@@ -39,7 +39,7 @@ public abstract class PropertyOwnerPointer extends 
NodePointer {
     private Object value = UNINITIALIZED;
 
     /**
-     * Create a new PropertyOwnerPointer.
+     * Constructs a new PropertyOwnerPointer.
      * 
      * @param parent pointer
      */
@@ -48,7 +48,7 @@ public abstract class PropertyOwnerPointer extends 
NodePointer {
     }
 
     /**
-     * Create a new PropertyOwnerPointer.
+     * Constructs a new PropertyOwnerPointer.
      * 
      * @param parent parent pointer
      * @param locale Locale
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/container/ContainerPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/container/ContainerPointer.java
index 8aae6ad..bbc26d9 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/container/ContainerPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/container/ContainerPointer.java
@@ -36,7 +36,7 @@ public class ContainerPointer extends NodePointer {
     private NodePointer valuePointer;
 
     /**
-     * Create a new ContainerPointer.
+     * Constructs a new ContainerPointer.
      * 
      * @param container Container object
      * @param locale    Locale
@@ -47,7 +47,7 @@ public class ContainerPointer extends NodePointer {
     }
 
     /**
-     * Create a new ContainerPointer.
+     * Constructs a new ContainerPointer.
      * 
      * @param parent    parent pointer
      * @param container Container object
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributeIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributeIterator.java
index c6a9c01..6aca108 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributeIterator.java
@@ -40,7 +40,7 @@ public class DOMAttributeIterator implements NodeIterator {
     private int position;
 
     /**
-     * Create a new DOMAttributeIterator.
+     * Constructs a new DOMAttributeIterator.
      * 
      * @param parent pointer
      * @param name   to test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributePointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributePointer.java
index 0ff240a..18eeb75 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributePointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMAttributePointer.java
@@ -34,7 +34,7 @@ public class DOMAttributePointer extends NodePointer {
     private final Attr attr;
 
     /**
-     * Create a new DOMAttributePointer.
+     * Constructs a new DOMAttributePointer.
      * 
      * @param parent pointer
      * @param attr   pointed
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNamespaceIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNamespaceIterator.java
index 6c041bf..b2b043e 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNamespaceIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNamespaceIterator.java
@@ -37,7 +37,7 @@ public class DOMNamespaceIterator implements NodeIterator {
     private int position;
 
     /**
-     * Create a new DOMNamespaceIterator.
+     * Constructs a new DOMNamespaceIterator.
      * 
      * @param parent parent pointer
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodeIterator.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodeIterator.java
index 1d3758a..7f9a22d 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodeIterator.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodeIterator.java
@@ -35,7 +35,7 @@ public class DOMNodeIterator implements NodeIterator {
     private int position;
 
     /**
-     * Create a new DOMNodeIterator.
+     * Constructs a new DOMNodeIterator.
      * 
      * @param parent    parent pointer
      * @param nodeTest  test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodePointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodePointer.java
index 33d00d7..dc39ac7 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodePointer.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/dom/DOMNodePointer.java
@@ -214,7 +214,7 @@ public class DOMNodePointer extends NodePointer {
     private NamespaceResolver localNamespaceResolver;
 
     /**
-     * Create a new DOMNodePointer.
+     * Constructs a new DOMNodePointer.
      * 
      * @param node   pointed at
      * @param locale Locale
@@ -225,7 +225,7 @@ public class DOMNodePointer extends NodePointer {
     }
 
     /**
-     * Create a new DOMNodePointer.
+     * Constructs a new DOMNodePointer.
      * 
      * @param node   pointed at
      * @param locale Locale
@@ -238,7 +238,7 @@ public class DOMNodePointer extends NodePointer {
     }
 
     /**
-     * Create a new DOMNodePointer.
+     * Constructs a new DOMNodePointer.
      * 
      * @param parent pointer
      * @param node   pointed
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dom/NamespacePointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/dom/NamespacePointer.java
index e8b858e..9d9e655 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/dom/NamespacePointer.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/dom/NamespacePointer.java
@@ -33,7 +33,7 @@ public class NamespacePointer extends NodePointer {
     private String namespaceURI;
 
     /**
-     * Create a new NamespacePointer.
+     * Constructs a new NamespacePointer.
      * 
      * @param parent parent pointer
      * @param prefix associated ns prefix.
@@ -44,7 +44,7 @@ public class NamespacePointer extends NodePointer {
     }
 
     /**
-     * Create a new NamespacePointer.
+     * Constructs a new NamespacePointer.
      * 
      * @param parent       parent pointer
      * @param prefix       associated ns prefix.
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPointer.java
index d227bc2..7efd9df 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPointer.java
@@ -47,7 +47,7 @@ public class DynaBeanPointer extends PropertyOwnerPointer {
     private final DynaBean dynaBean;
 
     /**
-     * Create a new DynaBeanPointer.
+     * Constructs a new DynaBeanPointer.
      * 
      * @param parent   pointer
      * @param name     is the name given to the first node
@@ -60,7 +60,7 @@ public class DynaBeanPointer extends PropertyOwnerPointer {
     }
 
     /**
-     * Create a new DynaBeanPointer.
+     * Constructs a new DynaBeanPointer.
      * 
      * @param name     is the name given to the first node
      * @param dynaBean pointed
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPropertyPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPropertyPointer.java
index 0eb25d8..1d86e00 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPropertyPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/DynaBeanPropertyPointer.java
@@ -41,7 +41,7 @@ public class DynaBeanPropertyPointer extends PropertyPointer {
     private String[] names;
 
     /**
-     * Create a new DynaBeanPropertyPointer.
+     * Constructs a new DynaBeanPropertyPointer.
      * 
      * @param parent   pointer
      * @param dynaBean pointed
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/StrictLazyDynaBeanPointerFactory.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/StrictLazyDynaBeanPointerFactory.java
index e557a01..8b88758 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/StrictLazyDynaBeanPointerFactory.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynabeans/StrictLazyDynaBeanPointerFactory.java
@@ -42,7 +42,7 @@ public class StrictLazyDynaBeanPointerFactory implements 
NodePointerFactory {
         private final LazyDynaBean lazyDynaBean;
 
         /**
-         * Create a new StrictLazyDynaBeanPointer instance.
+         * Constructs a new StrictLazyDynaBeanPointer instance.
          *
          * @param parent       pointer
          * @param name         is the name given to the first node
@@ -54,7 +54,7 @@ public class StrictLazyDynaBeanPointerFactory implements 
NodePointerFactory {
         }
 
         /**
-         * Create a new StrictLazyDynaBeanPointer instance.
+         * Constructs a new StrictLazyDynaBeanPointer instance.
          *
          * @param name         is the name given to the first node
          * @param lazyDynaBean pointed
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicAttributeIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicAttributeIterator.java
index 26eda90..b500322 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicAttributeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicAttributeIterator.java
@@ -28,7 +28,7 @@ import 
org.apache.commons.jxpath.ri.model.beans.PropertyOwnerPointer;
 public class DynamicAttributeIterator extends BeanAttributeIterator {
 
     /**
-     * Create a new DynamicAttributeIterator.
+     * Constructs a new DynamicAttributeIterator.
      * 
      * @param parent pointer
      * @param name   to match properties
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPointer.java
index df5278e..c35ca25 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPointer.java
@@ -40,7 +40,7 @@ public class DynamicPointer extends PropertyOwnerPointer {
     private final DynamicPropertyHandler handler;
 
     /**
-     * Create a new DynamicPointer.
+     * Constructs a new DynamicPointer.
      * 
      * @param parent  parent pointer
      * @param name    property name
@@ -55,7 +55,7 @@ public class DynamicPointer extends PropertyOwnerPointer {
     }
 
     /**
-     * Create a new DynamicPointer.
+     * Constructs a new DynamicPointer.
      * 
      * @param name    property name
      * @param bean    owning bean
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyIterator.java
index e86bbbf..aa7d9c6 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyIterator.java
@@ -28,7 +28,7 @@ import 
org.apache.commons.jxpath.ri.model.beans.PropertyOwnerPointer;
 public class DynamicPropertyIterator extends PropertyIterator {
 
     /**
-     * Create a new DynamicPropertyIterator
+     * Constructs a new DynamicPropertyIterator
      * 
      * @param pointer   PropertyOwnerPointer
      * @param name      String
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyPointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyPointer.java
index f46427f..e386742 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyPointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertyPointer.java
@@ -45,7 +45,7 @@ public class DynamicPropertyPointer extends PropertyPointer {
     private String requiredPropertyName;
 
     /**
-     * Create a new DynamicPropertyPointer.
+     * Constructs a new DynamicPropertyPointer.
      * 
      * @param parent  pointer
      * @param handler DynamicPropertyHandler
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMAttributeIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMAttributeIterator.java
index ef546af..e92963c 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMAttributeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMAttributeIterator.java
@@ -38,7 +38,7 @@ public class JDOMAttributeIterator implements NodeIterator {
     private int position;
 
     /**
-     * Create a new JDOMAttributeIterator.
+     * Constructs a new JDOMAttributeIterator.
      * 
      * @param parent pointer
      * @param name   test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespaceIterator.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespaceIterator.java
index 79c3c7f..de415d1 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespaceIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespaceIterator.java
@@ -39,7 +39,7 @@ public class JDOMNamespaceIterator implements NodeIterator {
     private int position;
 
     /**
-     * Create a new JDOMNamespaceIterator.
+     * Constructs a new JDOMNamespaceIterator.
      * 
      * @param parent the parent NodePointer.
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespacePointer.java
 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespacePointer.java
index 2237c42..acbb081 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespacePointer.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNamespacePointer.java
@@ -34,7 +34,7 @@ public class JDOMNamespacePointer extends NodePointer {
     private String namespaceURI;
 
     /**
-     * Create a new JDOMNamespacePointer.
+     * Constructs a new JDOMNamespacePointer.
      *
      * @param parent parent pointer
      * @param prefix ns prefix
@@ -45,7 +45,7 @@ public class JDOMNamespacePointer extends NodePointer {
     }
 
     /**
-     * Create a new JDOMNamespacePointer.
+     * Constructs a new JDOMNamespacePointer.
      *
      * @param parent       parent pointer
      * @param prefix       ns prefix
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodeIterator.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodeIterator.java
index 51f6b4c..4343408 100644
--- 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodeIterator.java
+++ 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodeIterator.java
@@ -40,7 +40,7 @@ public class JDOMNodeIterator implements NodeIterator {
     private Object child;
 
     /**
-     * Create a new JDOMNodeIterator.
+     * Constructs a new JDOMNodeIterator.
      * 
      * @param parent    pointer
      * @param nodeTest  test
diff --git 
a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodePointer.java 
b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodePointer.java
index e475e91..3e3d69d 100644
--- a/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodePointer.java
+++ b/src/main/java/org/apache/commons/jxpath/ri/model/jdom/JDOMNodePointer.java
@@ -231,7 +231,7 @@ public class JDOMNodePointer extends NodePointer {
     private NamespaceResolver localNamespaceResolver;
 
     /**
-     * Create a new JDOMNodePointer.
+     * Constructs a new JDOMNodePointer.
      *
      * @param parent NodePointer
      * @param node   pointed
@@ -242,7 +242,7 @@ public class JDOMNodePointer extends NodePointer {
     }
 
     /**
-     * Create a new JDOMNodePointer.
+     * Constructs a new JDOMNodePointer.
      *
      * @param node   node, like a bean.
      * @param locale Locale
@@ -253,7 +253,7 @@ public class JDOMNodePointer extends NodePointer {
     }
 
     /**
-     * Create a new JDOMNodePointer.
+     * Constructs a new JDOMNodePointer.
      *
      * @param node   pointed
      * @param locale Locale
diff --git 
a/src/main/java/org/apache/commons/jxpath/servlet/HttpSessionAndServletContext.java
 
b/src/main/java/org/apache/commons/jxpath/servlet/HttpSessionAndServletContext.java
index 591f405..bb8de8e 100644
--- 
a/src/main/java/org/apache/commons/jxpath/servlet/HttpSessionAndServletContext.java
+++ 
b/src/main/java/org/apache/commons/jxpath/servlet/HttpSessionAndServletContext.java
@@ -29,7 +29,7 @@ public class HttpSessionAndServletContext {
     private final ServletContext context;
 
     /**
-     * Create a new HttpSessionAndServletContext.
+     * Constructs a new HttpSessionAndServletContext.
      * 
      * @param session HttpSession
      * @param context ServletContext
diff --git 
a/src/main/java/org/apache/commons/jxpath/servlet/KeywordVariables.java 
b/src/main/java/org/apache/commons/jxpath/servlet/KeywordVariables.java
index d48518e..0e88447 100644
--- a/src/main/java/org/apache/commons/jxpath/servlet/KeywordVariables.java
+++ b/src/main/java/org/apache/commons/jxpath/servlet/KeywordVariables.java
@@ -37,7 +37,7 @@ public class KeywordVariables implements Variables {
     private final Object value;
 
     /**
-     * Create a new KeywordVariables.
+     * Constructs a new KeywordVariables.
      * 
      * @param keyword The keyword.
      * @param value  The value.
diff --git 
a/src/main/java/org/apache/commons/jxpath/servlet/PageScopeContext.java 
b/src/main/java/org/apache/commons/jxpath/servlet/PageScopeContext.java
index de12428..d5748fd 100644
--- a/src/main/java/org/apache/commons/jxpath/servlet/PageScopeContext.java
+++ b/src/main/java/org/apache/commons/jxpath/servlet/PageScopeContext.java
@@ -30,7 +30,7 @@ public class PageScopeContext {
     private final PageContext pageContext;
 
     /**
-     * Create a new PageScopeContext.
+     * Constructs a new PageScopeContext.
      * 
      * @param pageContext base
      */
diff --git 
a/src/main/java/org/apache/commons/jxpath/servlet/ServletRequestAndContext.java 
b/src/main/java/org/apache/commons/jxpath/servlet/ServletRequestAndContext.java
index 7632e25..639e76a 100644
--- 
a/src/main/java/org/apache/commons/jxpath/servlet/ServletRequestAndContext.java
+++ 
b/src/main/java/org/apache/commons/jxpath/servlet/ServletRequestAndContext.java
@@ -30,7 +30,7 @@ public class ServletRequestAndContext extends 
HttpSessionAndServletContext {
     private final ServletRequest request;
 
     /**
-     * Create a new ServletRequestAndContext.
+     * Constructs a new ServletRequestAndContext.
      *
      * @param request ServletRequest
      * @param context ServletContext
diff --git 
a/src/main/java/org/apache/commons/jxpath/util/BasicTypeConverter.java 
b/src/main/java/org/apache/commons/jxpath/util/BasicTypeConverter.java
index 85e303f..2f8ea1c 100644
--- a/src/main/java/org/apache/commons/jxpath/util/BasicTypeConverter.java
+++ b/src/main/java/org/apache/commons/jxpath/util/BasicTypeConverter.java
@@ -51,7 +51,7 @@ public class BasicTypeConverter implements TypeConverter {
         private List<Pointer> pointers;
 
         /**
-         * Create a new ValueNodeSet.
+         * Constructs a new ValueNodeSet.
          * 
          * @param values to return
          */
@@ -91,7 +91,7 @@ public class BasicTypeConverter implements TypeConverter {
         private final Object bean;
 
         /**
-         * Create a new ValuePointer.
+         * Constructs a new ValuePointer.
          * 
          * @param object value
          */
diff --git a/src/main/java/org/apache/commons/jxpath/util/KeyManagerUtils.java 
b/src/main/java/org/apache/commons/jxpath/util/KeyManagerUtils.java
index 3e832fa..5aba704 100644
--- a/src/main/java/org/apache/commons/jxpath/util/KeyManagerUtils.java
+++ b/src/main/java/org/apache/commons/jxpath/util/KeyManagerUtils.java
@@ -40,7 +40,7 @@ public class KeyManagerUtils {
         private final KeyManager delegate;
 
         /**
-         * Create a new SingleNodeExtendedKeyManager.
+         * Constructs a new SingleNodeExtendedKeyManager.
          * 
          * @param delegate KeyManager to wrap
          */
diff --git 
a/src/main/java/org/apache/commons/jxpath/util/ReverseComparator.java 
b/src/main/java/org/apache/commons/jxpath/util/ReverseComparator.java
index f57190b..b29f67e 100644
--- a/src/main/java/org/apache/commons/jxpath/util/ReverseComparator.java
+++ b/src/main/java/org/apache/commons/jxpath/util/ReverseComparator.java
@@ -32,7 +32,7 @@ public final class ReverseComparator implements Comparator, 
Serializable {
     public static final Comparator INSTANCE = new ReverseComparator();
 
     /**
-     * Create a new ReverseComparator.
+     * Constructs a new ReverseComparator.
      */
     private ReverseComparator() {
     }

Reply via email to