[jdom-interest] org.jdom.contrib.input.ResultSetBuilder

[Jason Hunter]

+1

I'd just ask that you update the Javadocs to be a bit more verbose on
what the name and namespace params do.  You can borrow from the existing
Javadocs.

-jh-

"Van Dooren, Damian" wrote:
> 
> I suggest adding the following 2 constructors to
> org.jdom.contrib.input.ResultSetBuilder purely for convience. I seem to
> always be setting the root and row names, so I just thought adding them and
> a Namespace to the constructor would simplify it.
> 
>     /**
>      * <p>
>      *   This sets up a <code>java.sql.ResultSet</code> to be built
>      *   as a <code>Document</code>.
>      * </p>
>      *
>      * @param rs <code>java.sql.ResultSet</code> to build
>      * @param rootName <code>String</code> the name to use.
>      * @param rowName <code>String</code> the name to use.
>      * @param ns <code>String</code> the namespace to use.
>      */
>     public ResultSetBuilder(ResultSet rs, String rootName, String rowName,
> Namespace ns) {
>         this(rs, rootName, rowName);
>         setNamespace(ns);
>     }
> 
>     /**
>      * <p>
>      *   This sets up a <code>java.sql.ResultSet</code> to be built
>      *   as a <code>Document</code>.
>      * </p>
>      *
>      * @param rs <code>java.sql.ResultSet</code> to build
>      * @param rootName <code>String</code> the name to use.
>      * @param rowName <code>String</code> the name to use.
>      */
>     public ResultSetBuilder(ResultSet rs, String rootName, String rowName) {
>         this(rs);
>         setRootName(rootName);
>         setRowName(rowName);
>     }
> 
> -----
> Damian Van Dooren
> Information Technology
> The Investment Centre
> (519) 672-4389 x718
> 
> _______________________________________________
> To control your jdom-interest membership:
> http://lists.denveronline.net/mailman/options/jdom-interest/youraddr@yourhost.com