[jdom-interest] getDocument() patch

Jason Hunter jhunter at acm.org
Mon Sep 25 13:53:21 PDT 2000


> http://code.werken.com/document_patch.txt
> 
> Synopsis:
> 
> Removed isRootElement boolean flag, and replaced with a Document

Just integrated the patch.

> It compiled, but I haven't really tested it.

Oops, didn't notice that remark until I integrated the patch.  I
wouldn't have integrated it had I seen it.  Just did some minimal
testing myself and it looked OK.

BTW, Brett wrote this when we started talking about this:

> I'm OK with this, but I don't have the time to make the changes. 
> There would need to be changes in Element.java, PartialList.java, 
> and Document.java, as far as I can tell.

You didn't make any changed to PartialList.  Brett, do you want to make
clear what changes you'd see?  The problem probably is that if you get
the list of content for the document itself and add an element to it,
there needs to be a setDocument() called!

> btw, diff -u now includes, by default, the ant-modified
> build scripts with EOL changes.  Blaugh.

Agreed.  Blaugh.

-jh-



More information about the jdom-interest mailing list