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

[cvs at jools.org]

Date:	Tuesday February 5, 2002 @ 3:03
Author:	cvs

Update of /home/cvspublic/jdom/src/java/org/jdom
In directory jools.org:/tmp/cvs-serv25738

Modified Files:
	DocType.java Element.java EntityRef.java Verifier.java 
Log Message:
Incorporating a great patch from elharo.  In his words...

--
Here's the diffs for the changes. They don't seem to break anything 
that wasn't broken before. Mostly this adds verification of the 
system and public IDs in both DocType and EntityRef, the root element 
name in DocType, and the entity name in EntityRef. Failing these 
causes the throwing of various runtime exceptions 
(IllegalDataException and IllegalNameexception) so this shouldn't 
break anybody's code.

It also allows null to be used for either system or public IDs in 
DocType to differentiate a missing ID from the empty string. This 
necessitated some small changes in some input and output classes.

I added one new Constructor to EntityRef to allow it to have a system 
ID without a public ID.

I also added a setElementName() method in DocType.

I added checkSystemLiteral() and checkPublicID() methods in Verifier.

I made checkXMLName() in Verifier public so EntityRef and DocType 
could use it. (They do allow colons in their names which 
checkElementName() forbids.) I don't think this will cause a problem, 
but if it does it could be package private.

Finally I fixed a few small errors I noticed in the doc comments here 
and there.
--

The only thing I didn't take was the change that allowed colons in XML
element names, since I'm thinking that probably causes more problems
than it solves.  I left elharo's comment in there, and we can always
make the one-line change later after the j-i discussion on the issue.

-jh-

===================================================================
File: no file DocType.java		Status: Needs Checkout

   Working revision:	1.18	Tue Feb  5 08:03:18 2002
   Repository revision:	1.18	/home/cvspublic/jdom/src/java/org/jdom/DocType.java,v

   Existing Tags:
	jdom_prefilter           	(revision: 1.15)
	jdom_1_0_b7              	(revision: 1.14)
	jdom_1_0_b6              	(revision: 1.7)
	start                    	(revision: 1.1.1.1)
	jdom                     	(branch: 1.1.1)

===================================================================
File: no file Element.java		Status: Needs Checkout

   Working revision:	1.107	Tue Feb  5 08:03:18 2002
   Repository revision:	1.107	/home/cvspublic/jdom/src/java/org/jdom/Element.java,v

   Existing Tags:
	jdom_prefilter           	(revision: 1.102)
	jdom_1_0_b7              	(revision: 1.96)
	jdom_1_0_b6              	(revision: 1.52)
	start                    	(revision: 1.1.1.1)
	jdom                     	(branch: 1.1.1)

===================================================================
File: no file EntityRef.java		Status: Needs Checkout

   Working revision:	1.6	Tue Feb  5 08:03:18 2002
   Repository revision:	1.6	/home/cvspublic/jdom/src/java/org/jdom/EntityRef.java,v

   Existing Tags:
	jdom_prefilter           	(revision: 1.3)
	jdom_1_0_b7              	(revision: 1.3)

===================================================================
File: no file Verifier.java		Status: Needs Checkout

   Working revision:	1.28	Tue Feb  5 08:03:18 2002
   Repository revision:	1.28	/home/cvspublic/jdom/src/java/org/jdom/Verifier.java,v

   Existing Tags:
	jdom_prefilter           	(revision: 1.23)
	jdom_1_0_b7              	(revision: 1.23)
	jdom_1_0_b6              	(revision: 1.19)
	start                    	(revision: 1.1.1.1)
	jdom                     	(branch: 1.1.1)