[jdom-interest] Bugs

[Brett McLaughlin]

> Leo Moot wrote:
> 
> 1. In "Document.java", method "public boolean
> removeProcessingInstructions(String target)"
>     the boolean deletedSome will never be set, change code to
> something like:
>       ...
>          if (((ProcessingInstruction)obj).getTarget().equals(target))
> {
>            deleteSome = true;
>            i.remove();
>          }
>       ...

Just committed it. Thanks!



> 
> 2. in "Element.Java", method "public boolean removeChildren()"
>     always returns true. Change code to something like:
>        ...
>          if (content == null)
>            return false;
> 
>          boolean deletedSome = false;
>          if (content != null) {
>            Iterator i = content.iterator();
>            while (i.hasNext()) {
>              Object obj = i.next();
>              if (obj instanceof Element) {
>                deletedSome = true;
>                i.remove();
>              }
>            }
>          }
> 
>          return deletedSome;
>          ...

This one got into the latest CVS, but thanks as well!


-Brett

> 
> Greetz & continue the good work!
> 
> --
> Leo Moot
> McKey & Partners
> 
> "Yea, and there shall come a time, when all other languages, save the
>  sacred C and C++, shall pass away, and yea Java and it's offspring
>  shall devour the land. They shall eat away at the sacred halls of all
> 
>  others and their child programs shall be found to be running
> everywhere!"
> 
> 
> __________________________________________________________
> 
> The views expressed in this posting are solely those of the author,
> and McKey & Partners does not endorse any of these views.
> McKey & Partners is not responsible for the accuracy or completeness
> of
> the information provided and assumes no duty to correct, expand upon,
> delete or update any of the information contained in this posting.
> ___________________________________________________________
> 

-- 
Brett McLaughlin, Enhydra Strategist
Lutris Technologies, Inc. 
1200 Pacific Avenue, Suite 300 
Santa Cruz, CA 95060 USA 
http://www.lutris.com
http://www.enhydra.org