[jdom-interest] Re: [jdom-commits] CVS update: jdom/src/java/org/jdom

[Jason Hunter]

I notice we used to have:

public Element addAttribute(String, String);
public Element addAttribute(Attribute);

Now we replaced it with:

public Element setAttribute(Attribute);

Wouldn't it make sense to reinstate:

public Element setAttribute(String, String);
public Element setAttribute(String, String, Namespace);

for convenience?

elt.setAttribute("name", "value");
vs
elt.setAttribute(new Attribute("name", "value"));

-jh-

bmclaugh at cvs.jdom.org wrote:
> 
> Date:   Monday June 18, 2001 @ 15:37
> Author: bmclaugh
> 
> Update of /home/cvspublic/jdom/src/java/org/jdom
> In directory www.nmemonix.com:/tmp/cvs-serv31108/src/java/org/jdom
> 
> Modified Files:
>         Element.java
> Log Message:
> Bringing in line the API with current practices. Deprecated addAttribute(String name, String prefix, String value). setAttribute() should be used. Also added removeContent(CDATA cdata), which was simply overlooked.
> ===================================================================
> File: no file Element.java              Status: Needs Checkout
> 
>    Working revision:    1.86    Mon Jun 18 14:37:10 2001
>    Repository revision: 1.86    /home/cvspublic/jdom/src/java/org/jdom/Element.java,v
> 
>    Existing Tags:
>         start                           (revision: 1.1.1.1)
>         jdom                            (branch: 1.1.1)
> 
> _______________________________________________
> To control your jdom-commits membership:
> http://lists.denveronline.net/mailman/options/jdom-commits/youraddr@yourhost.com