[Repost] Re: [Sax-devel] [PATCH]: A couple fixes to org/xml/sax/package.html

[David M. <dm...@at...>]

[Originally posted earlier today, but it didn't get through.]

Karl Waclawek wrote:

>>2) Specify that the use-entity-resolver2 is read-write with a default 
>>value of true, instead of read-only with no default value. The description 
>>of EntityResolver2 in CVS says: "If a SAX application requires the 
>>customized handling which this interface defines for external entities, it 
>>must ensure that it uses an XMLReader with the 
>>http://xml.org/sax/features/use-entity-resolver2 feature flag set to true 
>>(which is its default value when the feature is recognized). If that flag 
>>is unrecognized, or its value is false, or the resolver does not implement 
>>this interface, then only the EntityResolver method will be used." 
>>Therefore this feature has a default value of true (when it is 
>>recognized). Since the default value is true, the value of the feature 
>>could only ever be false if it were read-write. Parsers which do not 
>>support setting this feature to false would throw a 
>>SAXNotSupportedException.
> 
> Alternatively, you could fix the description of EntityResolver2
> with the attached patch and leave it read-only.

Thanks for the patch.  Before I commit, will we need a patch to package.html
(with Michael's recent changes) for consistency?


All the best,


David