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

[Brett McLaughlin]

> 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"));

I dunno. I guess it's marginally more convenient, but after having just
documented all the methods in my 2nd edition's appendix, I like brevity ;-)

I'm +0

-Brett

>
> -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
> _______________________________________________
> To control your jdom-interest membership:
>
http://lists.denveronline.net/mailman/options/jdom-interest/youraddr@yourhos
t.com