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

SourceForge.net noreply at sourceforge.net
Thu Jan 23 22:00:44 EST 2003


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: 7
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: Fred L. Drake, Jr. (fdrake)
Date: 2003-01-24 01:00

Message:
Logged In: YES 
user_id=3066

Jeremy:  Thanks for the update with regard to XML NS 1.1.  I
agree we don't need to worry about that yet; it's more
important to fix bugs with regard to XML 1.0 and the first
version of the namespaces spec.  See my previous note.

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

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

Message:
Logged In: YES 
user_id=3066

Ok, *now* I'm not happy.  I've done my "cvs up", and the
patch doesn't appear to have fixed the problem.  Perhaps I
don't have proper tests yet?  I've attached them to this
issue report.  I expected these tests to pass with current
CVS since Karl checked in Jeremy's patch.

I'll have to deal with this tomorrow; I'm too tired to go
trawling through this just yet.  The patched looked fine on
the surface; did it work for you?

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

Comment By: Jeremy Kloth (jkloth)
Date: 2003-01-24 00:41

Message:
Logged In: YES 
user_id=38980

As far as the bug in the namespace spec goes, they addressed just 
his particular issue with XML NS 1.1.  However that would required XML 
1.1 processing which I'm not sure is something that is needed right 
now (although it is in canidate rec). 

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

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

Message:
Logged In: YES 
user_id=3066

What Karl didn't say was that he checked in the patch as
lib/xmlparse.c revision 1.105.  ;-)  I'll just avoid doing a
"cvs up" until the test is done -- should be soon.

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

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



More information about the Expat-bugs mailing list