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-jelly.git


The following commit(s) were added to refs/heads/master by this push:
     new 30eca265 Javadoc
30eca265 is described below

commit 30eca2650e5fcf494085c151f7406e7046cace24
Author: Gary D. Gregory <[email protected]>
AuthorDate: Sun May 4 10:25:06 2025 -0400

    Javadoc
    
    - Use final
    - Remove useless semi-colons
---
 .../main/java/org/apache/commons/jelly/Tag.java    |  55 +++++---
 .../org/apache/commons/jelly/parser/XMLParser.java |  25 ++--
 .../apache/commons/jelly/servlet/JellyServlet.java |   4 +-
 .../commons/jelly/servlet/JellyServletContext.java |  18 +--
 .../apache/commons/jelly/tags/core/IncludeTag.java |  39 +++---
 .../apache/commons/jelly/tags/core/UseBeanTag.java | 147 +++++++++------------
 6 files changed, 132 insertions(+), 156 deletions(-)

diff --git a/core/src/main/java/org/apache/commons/jelly/Tag.java 
b/core/src/main/java/org/apache/commons/jelly/Tag.java
index 99df141c..84dc3d2d 100644
--- a/core/src/main/java/org/apache/commons/jelly/Tag.java
+++ b/core/src/main/java/org/apache/commons/jelly/Tag.java
@@ -18,66 +18,81 @@
 package org.apache.commons.jelly;
 
 /**
- * <p><code>Tag</code> represents a Jelly custom tag.
- * A Tag is only ever used by a single thread so that Tag developers do not
- * need to concern themselves with mutli-threading issues when writing a Tag.
- * A Tag is created per custom tag in a script, per invocation.
- * So there is no need to worry about pooling errors like those caused
- * in JSP 1.x.(</p>
+ * <code>Tag</code> represents a Jelly custom tag. A Tag is only ever used by 
a single thread so that Tag developers do not need to concern themselves with
+ * mutli-threading issues when writing a Tag. A Tag is created per custom tag 
in a script, per invocation. So there is no need to worry about pooling errors
+ * like those caused in JSP 1.x.(
  */
 public interface Tag {
 
     /**
-     * @return the parent of this tag
+     * Gets the parent of this tag.
+     *
+     * @return the parent of this tag.
      */
     public Tag getParent();
 
     /**
-     * Sets the parent of this tag
+     * Sets the parent of this tag.
+     *
+     * @param parent the parent of this tag.
      */
     public void setParent(Tag parent);
-    
+
     /**
-     * Returns the TagLibrary - will be null if this is an unrecognized tag 
-     * (ie a StaticTag) 
-     * @return
+     * Gets the TagLibrary, null if this is an unrecognized tag (ie a 
StaticTag)
+     *
+     * @return the TagLibrary.
      */
     public TagLibrary getTagLib();
-    
+
     /**
-     * Sets the tag library
-     * @param tagLibrary
+     * Sets the tag library.
+     *
+     * @param tagLibrary the tag library.
      */
     public void setTagLib(TagLibrary tagLibrary);
 
     /**
-     * @return the body of the tag
+     * @return the body of the tag.
      */
     public Script getBody();
 
     /**
-     * Sets the body of the tag
+     * Sets the body of the tag.
+     *
+     * @param body the body of the tag.
      */
     public void setBody(Script body);
 
     /**
-     * Gets the context in which the tag will be run
+     * Gets the context in which the tag will be run.
+     *
+     * @return the context in which the tag will be run
      */
     public JellyContext getContext();
 
     /**
-     * Sets the context in which the tag will be run
+     * Sets the context in which the tag will be run.
+     *
+     * @param context the context in which the tag will be run.
+     * @throws JellyTagException Thrown on error.
      */
     public void setContext(JellyContext context) throws JellyTagException;
 
     /**
      * Evaluates this tag after all the tags properties have been initialized.
+     *
+     * @param output output stream.
+     * @throws MissingAttributeException Thrown on error.
+     * @throws JellyTagException Thrown on error.
      */
     public void doTag(XMLOutput output) throws MissingAttributeException, 
JellyTagException;
 
     /**
      * A helper method to invoke this tags body
+     *
+     * @param output XML output stream.
+     * @throws JellyTagException Thrown on error.
      */
     public void invokeBody(XMLOutput output) throws JellyTagException;
-
 }
diff --git a/core/src/main/java/org/apache/commons/jelly/parser/XMLParser.java 
b/core/src/main/java/org/apache/commons/jelly/parser/XMLParser.java
index 8396498b..245a2d27 100644
--- a/core/src/main/java/org/apache/commons/jelly/parser/XMLParser.java
+++ b/core/src/main/java/org/apache/commons/jelly/parser/XMLParser.java
@@ -224,6 +224,7 @@ public class XMLParser extends DefaultHandler {
      * the root element from the object stack (if any).
      *
      * @param file File containing the XML data to be parsed
+     * @return The script.
      * @throws IOException if an input/output error occurs
      * @throws SAXException if a parsing exception occurs
      */
@@ -236,6 +237,7 @@ public class XMLParser extends DefaultHandler {
      * the root element from the object stack (if any).
      *
      * @param url URL containing the XML data to be parsed
+     * @return The script.
      * @throws IOException if an input/output error occurs
      * @throws SAXException if a parsing exception occurs
      */
@@ -254,6 +256,7 @@ public class XMLParser extends DefaultHandler {
      * Returns the root element from the object stack (if any).
      *
      * @param input Input source containing the XML data to be parsed
+     * @return The script.
      * @throws IOException if an input/output error occurs
      * @throws SAXException if a parsing exception occurs
      */
@@ -272,7 +275,7 @@ public class XMLParser extends DefaultHandler {
      * to resolve any relative paths inside a DTD.)
      *
      * @param input  Input stream containing the XML data to be parsed
-     * @return
+     * @return The script.
      * @throws IOException
      *                   if an input/output error occurs
      * @throws SAXException
@@ -293,7 +296,7 @@ public class XMLParser extends DefaultHandler {
      * to resolve any relative paths inside a DTD.)
      *
      * @param reader Reader containing the XML data to be parsed
-     * @return
+     * @return The script.
      * @throws IOException
      *                   if an input/output error occurs
      * @throws SAXException
@@ -311,6 +314,7 @@ public class XMLParser extends DefaultHandler {
      * Returns the root element from the object stack (if any).
      *
      * @param uri URI containing the XML data to be parsed
+     * @return The script.
      * @throws IOException if an input/output error occurs
      * @throws SAXException if a parsing exception occurs
      */
@@ -327,6 +331,7 @@ public class XMLParser extends DefaultHandler {
      * go dynamically as the document is parsed.
      *
      * @param prefix Prefix to look up
+     * @return The matching prefix.
      */
     public String findNamespaceURI(String prefix) {
         ArrayDeque stack = (ArrayDeque) namespaces.get(prefix);
@@ -341,8 +346,6 @@ public class XMLParser extends DefaultHandler {
         }
     }
 
-    // Properties
-    //-------------------------------------------------------------------------
     public JellyContext getContext() {
         return context;
     }
@@ -490,7 +493,7 @@ public class XMLParser extends DefaultHandler {
      * the default implementation defers to the TagLibrary to create the 
Expression
      * @param attributeName
      * @param value
-     * @return
+     * @return The parsed exception.
      * @throws JellyException
      */
     public Expression createExpression(TagScript script, String attributeName, 
String value) throws JellyException {
@@ -850,7 +853,7 @@ public class XMLParser extends DefaultHandler {
     @Override
     public void ignorableWhitespace(char buffer[], int start, int len)
         throws SAXException {
-        ; // No processing required
+        // No processing required
     }
 
     /**
@@ -863,7 +866,7 @@ public class XMLParser extends DefaultHandler {
     @Override
     public void processingInstruction(String target, String data)
         throws SAXException {
-        ; // No processing is required
+        // No processing is required
     }
 
     /**
@@ -884,12 +887,9 @@ public class XMLParser extends DefaultHandler {
      */
     @Override
     public void skippedEntity(String name) throws SAXException {
-        ; // No processing required
+        // No processing required
     }
 
-    // DTDHandler interface
-    //-------------------------------------------------------------------------
-
     /**
      * Receive notification of a notation declaration event.
      *
@@ -917,9 +917,6 @@ public class XMLParser extends DefaultHandler {
         String notation) {
     }
 
-    // ErrorHandler interface
-    //-------------------------------------------------------------------------
-
     /**
      * Forward notification of a parsing error to the application supplied
      * error handler, if any, otherwise throw a SAXException with the error.
diff --git 
a/core/src/main/java/org/apache/commons/jelly/servlet/JellyServlet.java 
b/core/src/main/java/org/apache/commons/jelly/servlet/JellyServlet.java
index 4d5850ce..443cdbd0 100644
--- a/core/src/main/java/org/apache/commons/jelly/servlet/JellyServlet.java
+++ b/core/src/main/java/org/apache/commons/jelly/servlet/JellyServlet.java
@@ -90,7 +90,7 @@ public class JellyServlet extends HttpServlet {
      * See org.apache.velocity.servlet.VelocityServlet#createContext
      * @param req
      * @param res
-     * @return
+     * @return a new context.
      */
     protected JellyContext createContext(
         HttpServletRequest req,
@@ -113,7 +113,7 @@ public class JellyServlet extends HttpServlet {
      *
      * See org.apache.velocity.servlet.VelocityServlet#getTemplate
      * @param req
-     * @return
+     * @return a URL.
      * @throws MalformedURLException
      */
     protected URL getScript(HttpServletRequest req)
diff --git 
a/core/src/main/java/org/apache/commons/jelly/servlet/JellyServletContext.java 
b/core/src/main/java/org/apache/commons/jelly/servlet/JellyServletContext.java
index dbfd0d74..3b987982 100644
--- 
a/core/src/main/java/org/apache/commons/jelly/servlet/JellyServletContext.java
+++ 
b/core/src/main/java/org/apache/commons/jelly/servlet/JellyServletContext.java
@@ -35,40 +35,40 @@ public class JellyServletContext extends JellyContext {
     public JellyServletContext() {
     }
 
-    public JellyServletContext(ServletContext ctx) {
-        super();
+    public JellyServletContext(final ServletContext ctx) {
         this.ctx = ctx;
     }
 
-    public JellyServletContext(JellyContext parent, ServletContext ctx) {
+    public JellyServletContext(final JellyContext parent, final ServletContext 
ctx) {
         super(parent);
         this.ctx = ctx;
     }
 
     /**
      * Allow access of relative URIs when performing &lt;j:include&gt;.
+     *
      * @param s
-     * @return
+     * @return The {@code InputStream} returned to the servlet, or {@code 
null} if no resource exists at the specified path
      * @throws MalformedURLException
      */
     @Override
-    public URL getResource(String s) throws MalformedURLException {
+    public URL getResource(final String s) throws MalformedURLException {
         return ctx.getResource(s);
     }
 
     /**
      * Allow access of relative URIs when performing &lt;j:include&gt;.
+     *
      * @param s
-     * @return
+     * @return The {@code InputStream} returned to the servlet, or {@code 
null} if no resource exists at the specified path.
      */
     @Override
-    public InputStream getResourceAsStream(String s) {
+    public InputStream getResourceAsStream(final String s) {
         return ctx.getResourceAsStream(s);
     }
 
     @Override
-    protected JellyContext createChildContext()
-    {
+    protected JellyContext createChildContext() {
         return new JellyServletContext(this, ctx);
     }
 }
diff --git 
a/core/src/main/java/org/apache/commons/jelly/tags/core/IncludeTag.java 
b/core/src/main/java/org/apache/commons/jelly/tags/core/IncludeTag.java
index f0670624..f9ca9e18 100644
--- a/core/src/main/java/org/apache/commons/jelly/tags/core/IncludeTag.java
+++ b/core/src/main/java/org/apache/commons/jelly/tags/core/IncludeTag.java
@@ -25,14 +25,13 @@ import org.apache.commons.jelly.MissingAttributeException;
 import org.apache.commons.jelly.TagSupport;
 import org.apache.commons.jelly.XMLOutput;
 
-/** A tag which conditionally evaluates its body based on some condition
-  */
-
+/**
+ * A tag which conditionally evaluates its body based on some condition
+ */
 public class IncludeTag extends TagSupport {
 
     private String uri;
     private File file;
-
     private boolean shouldExport;
     private boolean shouldInherit;
 
@@ -41,7 +40,7 @@ public class IncludeTag extends TagSupport {
         this.shouldInherit = true;
     }
 
-    public void setInherit(String inherit) {
+    public void setInherit(final String inherit) {
         if ("true".equals(inherit)) {
             this.shouldInherit = true;
         } else {
@@ -49,7 +48,7 @@ public class IncludeTag extends TagSupport {
         }
     }
 
-    public void setExport(String export) {
+    public void setExport(final String export) {
         if ("true".equals(export)) {
             this.shouldExport = true;
         } else {
@@ -66,30 +65,28 @@ public class IncludeTag extends TagSupport {
     }
 
     /**
-     * @return
+     * Gets the file.
+     *
+     * @return the file.
      */
     public File getFile() {
         return file;
     }
 
     /**
-     * Sets the file to be included which is either an absolute file or a file
-     * relative to the current directory
+     * Sets the file to be included which is either an absolute file or a file 
relative to the current directory
+     *
+     * @param file A file..
      */
-    public void setFile(File file) {
+    public void setFile(final File file) {
         this.file = file;
     }
 
-    // Tag interface
-    //-------------------------------------------------------------------------
     @Override
-    public void doTag(XMLOutput output)
-        throws MissingAttributeException, JellyTagException {
-
+    public void doTag(final XMLOutput output) throws 
MissingAttributeException, JellyTagException {
         if (uri == null && file == null) {
             throw new MissingAttributeException("uri");
         }
-
         // we need to create a new JellyContext of the URI
         // take off the script name from the URL
         String text = null;
@@ -97,21 +94,17 @@ public class IncludeTag extends TagSupport {
             if (uri != null) {
                 text = uri;
                 context.runScript(uri, output, isExport(), isInherit());
-            }
-            else {
+            } else {
                 text = file.toString();
                 context.runScript(file, output, isExport(), isInherit());
             }
-        }
-        catch (JellyException e) {
+        } catch (final JellyException e) {
             throw new JellyTagException("could not include jelly script: " + 
text + ". Reason: " + e, e);
         }
     }
 
-    // Properties
-    //-------------------------------------------------------------------------
     /** Sets the URI (relative URI or absolute URL) for the script to 
evaluate. */
-    public void setUri(String uri) {
+    public void setUri(final String uri) {
         this.uri = uri;
     }
 }
diff --git 
a/core/src/main/java/org/apache/commons/jelly/tags/core/UseBeanTag.java 
b/core/src/main/java/org/apache/commons/jelly/tags/core/UseBeanTag.java
index 0849930a..ad0f0adc 100644
--- a/core/src/main/java/org/apache/commons/jelly/tags/core/UseBeanTag.java
+++ b/core/src/main/java/org/apache/commons/jelly/tags/core/UseBeanTag.java
@@ -14,6 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.apache.commons.jelly.tags.core;
 
 import java.lang.reflect.InvocationTargetException;
@@ -33,14 +34,11 @@ import org.apache.commons.jelly.impl.BeanSource;
 import org.apache.commons.jelly.util.ClassLoaderUtils;
 
 /**
- * A tag which instantiates an instance of the given class
- * and then sets the properties on the bean.
- * The class can be specified via a {@link Class} instance or
- * a String which will be used to load the class using either the current
- * thread's context class loader or the class loader used to load this
- * Jelly library.
+ * A tag which instantiates an instance of the given class and then sets the 
properties on the bean. The class can be specified via a {@link Class} instance 
or
+ * a String which will be used to load the class using either the current 
thread's context class loader or the class loader used to load this Jelly 
library.
  *
  * This tag can be used it as follows,
+ *
  * <pre>
  * &lt;j:useBean var="person" class="com.acme.Person" name="James" 
location="${loc}"/&gt;
  * &lt;j:useBean var="order" class="${orderClass}" amount="12" 
price="123.456"/&gt;
@@ -50,20 +48,15 @@ public class UseBeanTag extends MapTagSupport implements 
BeanSource {
 
     /** The current bean instance */
     private Object bean;
-
     /** The default class to use if no Class is specified */
     private Class defaultClass;
-
     /**
-     * a Set of Strings of property names to ignore (remove from the
-     * Map of attributes before passing to ConvertUtils)
+     * a Set of Strings of property names to ignore (remove from the Map of 
attributes before passing to ConvertUtils)
      */
     private Set ignoreProperties;
-
     /**
-     * If this tag finds an attribute in the XML that's not
-     * ignored by {@link #ignoreProperties} and isn't a
-     * bean property, should it throw an exception?
+     * If this tag finds an attribute in the XML that's not ignored by {@link 
#ignoreProperties} and isn't a bean property, should it throw an exception?
+     *
      * @see #setIgnoreUnknownProperties(boolean)
      */
     private boolean ignoreUnknownProperties = false;
@@ -71,12 +64,11 @@ public class UseBeanTag extends MapTagSupport implements 
BeanSource {
     public UseBeanTag() {
     }
 
-    public UseBeanTag(Class defaultClass) {
+    public UseBeanTag(final Class defaultClass) {
         this.defaultClass = defaultClass;
     }
-
     // BeanSource interface
-    //-------------------------------------------------------------------------
+    // 
-------------------------------------------------------------------------
 
     /**
      * @return the bean that has just been created
@@ -87,124 +79,105 @@ public class UseBeanTag extends MapTagSupport implements 
BeanSource {
     }
 
     // Tag interface
-    //-------------------------------------------------------------------------
+    // 
-------------------------------------------------------------------------
     @Override
-    public void doTag(XMLOutput output) throws JellyTagException {
-        Map attributes = getAttributes();
-        String var = (String) attributes.get( "var" );
-        Object classObject = attributes.get( "class" );
+    public void doTag(final XMLOutput output) throws JellyTagException {
+        final Map attributes = getAttributes();
+        final String var = (String) attributes.get("var");
+        final Object classObject = attributes.get("class");
         addIgnoreProperty("class");
         addIgnoreProperty("var");
-
         try {
             // this method could return null in derived classes
-            Class theClass = convertToClass(classObject);
-
+            final Class theClass = convertToClass(classObject);
             bean = newInstance(theClass, attributes, output);
             setBeanProperties(bean, attributes);
-
             // invoke body which could result in other properties being set
             invokeBody(output);
-
             processBean(var, bean);
-        }
-        catch (ClassNotFoundException e) {
+        } catch (final ClassNotFoundException e) {
             throw new JellyTagException(e);
         }
     }
-
     // Implementation methods
-    //-------------------------------------------------------------------------
+    // 
-------------------------------------------------------------------------
 
     /**
      * Allow derived classes to programmatically set the bean
      */
-    protected void setBean(Object bean) {
+    protected void setBean(final Object bean) {
         this.bean = bean;
     }
 
     /**
-     * Attempts to convert the given object to a Class instance.
-     * If the classObject is already a Class it will be returned
-     * otherwise it will be converted to a String and loaded
-     * using the default class loading mechanism.
+     * Attempts to convert the given object to a Class instance. If the 
classObject is already a Class it will be returned otherwise it will be 
converted to a
+     * String and loaded using the default class loading mechanism.
      */
-    protected Class convertToClass(Object classObject)
-    throws MissingAttributeException, ClassNotFoundException {
+    protected Class convertToClass(final Object classObject) throws 
MissingAttributeException, ClassNotFoundException {
         if (classObject instanceof Class) {
             return (Class) classObject;
         }
-        else if ( classObject == null ) {
-            Class theClass = getDefaultClass();
+        if (classObject == null) {
+            final Class theClass = getDefaultClass();
             if (theClass == null) {
                 throw new MissingAttributeException("class");
             }
             return theClass;
-        }
-        else {
-            String className = classObject.toString();
+        } else {
+            final String className = classObject.toString();
             return loadClass(className);
         }
     }
 
     /**
-     * Loads the given class using the default class loading mechanism
-     * which is to try use the current Thread's context class loader first
-     * otherwise use the class loader which loaded this class.
+     * Loads the given class using the default class loading mechanism which 
is to try use the current Thread's context class loader first otherwise use the
+     * class loader which loaded this class.
      */
-    protected Class loadClass(String className) throws ClassNotFoundException {
+    protected Class loadClass(final String className) throws 
ClassNotFoundException {
         return ClassLoaderUtils.loadClass(className, getClass());
     }
 
     /**
-     * Creates a new instance of the given class, which by default will invoke 
the
-     * default constructor.
-     * Derived tags could do something different here.
+     * Creates a new instance of the given class, which by default will invoke 
the default constructor. Derived tags could do something different here.
      */
-    protected Object newInstance(Class theClass, Map attributes, XMLOutput 
output)
-    throws JellyTagException {
+    protected Object newInstance(final Class theClass, final Map attributes, 
final XMLOutput output) throws JellyTagException {
         try {
             return theClass.getConstructor().newInstance();
-        } catch (ReflectiveOperationException e) {
+        } catch (final ReflectiveOperationException e) {
             throw new JellyTagException(e.toString());
         }
     }
 
     /**
-     * Sets the properties on the bean. Derived tags could implement some 
custom
-     * type conversion etc.
+     * Sets the properties on the bean. Derived tags could implement some 
custom type conversion etc.
      * <p>
      * This method ignores all property names in the Set returned by {@link 
#getIgnorePropertySet()}.
      * </p>
      */
-    protected void setBeanProperties(Object bean, Map attributes) throws 
JellyTagException {
-        Map attrsToUse = new HashMap(attributes);
+    protected void setBeanProperties(final Object bean, final Map attributes) 
throws JellyTagException {
+        final Map attrsToUse = new HashMap(attributes);
         attrsToUse.keySet().removeAll(getIgnorePropertySet());
-
         validateBeanProperties(bean, attrsToUse);
-
         try {
             BeanUtils.populate(bean, attrsToUse);
-        } catch (IllegalAccessException e) {
-            throw new JellyTagException("could not set the properties of the 
bean", e);
-        } catch (InvocationTargetException e) {
+        } catch (final IllegalAccessException | InvocationTargetException e) {
             throw new JellyTagException("could not set the properties of the 
bean", e);
         }
     }
 
     /**
-     * If {@link #isIgnoreUnknownProperties()} returns true, make sure that
-     * every non-ignored ({@link #addIgnoreProperty(String)}) property
-     * matches a writable property on the target bean.
-     * @param bean the bean to validate
+     * If {@link #isIgnoreUnknownProperties()} returns true, make sure that 
every non-ignored ({@link #addIgnoreProperty(String)}) property matches a 
writable
+     * property on the target bean.
+     *
+     * @param bean       the bean to validate
      * @param attributes the list of properties to validate
      * @throws JellyTagException when a property is not writeable
      */
-    protected void validateBeanProperties(Object bean, Map attributes) throws 
JellyTagException {
+    protected void validateBeanProperties(final Object bean, final Map 
attributes) throws JellyTagException {
         if (!isIgnoreUnknownProperties()) {
-            for (Iterator i=attributes.keySet().iterator();i.hasNext();) {
-                String attrName = (String)i.next();
-                if (! PropertyUtils.isWriteable(bean, attrName)) {
+            for (final Iterator i = attributes.keySet().iterator(); 
i.hasNext();) {
+                final String attrName = (String) i.next();
+                if (!PropertyUtils.isWriteable(bean, attrName)) {
                     throw new JellyTagException("No bean property found: " + 
attrName);
                 }
             }
@@ -212,16 +185,14 @@ public class UseBeanTag extends MapTagSupport implements 
BeanSource {
     }
 
     /**
-     * By default this will export the bean using the given variable if it is 
defined.
-     * This Strategy method allows derived tags to process the beans in 
different ways
-     * such as to register this bean with its parent tag etc.
+     * By default this will export the bean using the given variable if it is 
defined. This Strategy method allows derived tags to process the beans in
+     * different ways such as to register this bean with its parent tag etc.
      */
-    protected void processBean(String var, Object bean) throws 
JellyTagException {
+    protected void processBean(final String var, final Object bean) throws 
JellyTagException {
         if (var != null) {
             context.setVariable(var, bean);
-        }
-        else {
-            ArgTag parentArg = (ArgTag)(findAncestorWithClass(ArgTag.class));
+        } else {
+            final ArgTag parentArg = (ArgTag) 
findAncestorWithClass(ArgTag.class);
             if (null != parentArg) {
                 parentArg.setValue(bean);
             }
@@ -236,42 +207,42 @@ public class UseBeanTag extends MapTagSupport implements 
BeanSource {
     }
 
     /**
-     * Adds a name to the Set of property names that will be skipped when 
setting
-     * bean properties. In other words, names added here won't be set into the 
bean
+     * Adds a name to the Set of property names that will be skipped when 
setting bean properties. In other words, names added here won't be set into the 
bean
      * if they're present in the attribute Map.
+     *
      * @param name
      */
-    protected void addIgnoreProperty(String name) {
+    protected void addIgnoreProperty(final String name) {
         getIgnorePropertySet().add(name);
     }
 
     /**
-     * @return the Set of property names that should be ignored when setting 
the
-     * properties of the bean.
+     * @return the Set of property names that should be ignored when setting 
the properties of the bean.
      */
     protected Set getIgnorePropertySet() {
         if (ignoreProperties == null) {
             ignoreProperties = new HashSet();
         }
-
         return ignoreProperties;
     }
 
     /**
+     * Tests If this tag finds an attribute in the XML that's not ignored by 
{@link #ignoreProperties} and isn't a bean property, should it throw an 
exception?
+     *
+     * @see #setIgnoreUnknownProperties(boolean)
      * @see #setIgnoreUnknownProperties(boolean)
-     * @return
+     * @return whether to ignore unknown properties.
      */
     public boolean isIgnoreUnknownProperties() {
         return ignoreUnknownProperties;
     }
 
     /**
-     * If this tag finds an attribute in the XML that's not
-     * ignored by {@link #ignoreProperties} and isn't a
-     * bean property, should it throw an exception?
+     * If this tag finds an attribute in the XML that's not ignored by {@link 
#ignoreProperties} and isn't a bean property, should it throw an exception?
+     *
      * @param ignoreUnknownProperties Sets {@link #ignoreUnknownProperties}.
      */
-    public void setIgnoreUnknownProperties(boolean ignoreUnknownProperties) {
+    public void setIgnoreUnknownProperties(final boolean 
ignoreUnknownProperties) {
         this.ignoreUnknownProperties = ignoreUnknownProperties;
     }
 }

Reply via email to