[jdom-interest] Element.setAttribute(s)() B7 multiple bugs?
    Alex Rosen 
    arosen at silverstream.com
       
    Wed Aug  1 08:01:20 PDT 2001
    
    
  
> Here's the pertinent code:
>
>     public Element setAttributes(List attributes) {
>         // XXX Verify attributes are all parentless first
>         this.attributes = attributes;
>         return this;
>     }
Hmmm... On an unrelated topic: Doesn't this need some work? First of all, it
doesn't setParent() on the passed-in Attributes. Second, the List that you
pass in is now live - changes you make to the List will affect the
attributes of this element now. Which is different from setContent(List).
And, for our other live lists (the ones we return), don't we check to make
sure that subsequently added objects are of the correct type?
Alex
    
    
More information about the jdom-interest
mailing list