[Expat-bugs] [ expat-Patches-673791 ] Error with xmlns:prefix=""

[SourceForge.net]

Patches item #673791, was opened at 2003-01-23 22:47
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=310127&aid=673791&group_id=10127

Category: None
Group: None
Status: Open
Resolution: Accepted
Priority: 6
Submitted By: Jeremy Kloth (jkloth)
Assigned to: Fred L. Drake, Jr. (fdrake)
>Summary: Error with xmlns:prefix="" 

Initial Comment:
When processing a document in namespace mode, the 
following document should be in error (according to XML 
Namespaces 1.0 Section 2): 
 
<?xml version='1.0'?> 
<docelem xmlns:pre=""/> 
 
However, it is being processed just fine. 
 
With the attached patch, this error is properly reported (only 
when namespace processing is enabled). 
 
I hope this can get in before the 1.95.6 release. 
 
Thanks, 
Jeremy Kloth 

----------------------------------------------------------------------

>Comment By: Karl Waclawek (kwaclaw)
Date: 2003-01-24 00:08

Message:
Logged In: YES 
user_id=290026

Sorry, Fred.

I jumped the gun - wanted to finish it before going to bed.
It really looks OK, so I don't feel I risked too much.

----------------------------------------------------------------------

Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2003-01-24 00:03

Message:
Logged In: YES 
user_id=3066

I think you're right.  I haven't looked at the patch yet,
but the spec is (almost) clear about this.

I will note that I consider this more a bug in the namespace
spec than a desirable feature or constraint to test:  It
makes more sense (to me) to allow the use of xmlns:prefix=''
to "un-declare" the prefix (possibly making it an error of
some sort to un-declare a prefix that doesn't have an active
declaration).  But that's my opinion, not the letter of the
spec.

I'll mark this as accepted (for concept), and work up a test
case before committing anything.

----------------------------------------------------------------------

Comment By: Karl Waclawek (kwaclaw)
Date: 2003-01-23 23:37

Message:
Logged In: YES 
user_id=290026

Yes, it seems the NS specs do not allow a prefix name with 
an empty URI. This patch looks OK to me - if Fred agrees, I 
can commit it. 

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=310127&aid=673791&group_id=10127