You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@santuario.apache.org by mu...@apache.org on 2007/09/13 23:29:24 UTC

svn commit: r575466 - /xml/security/branches/stax_jsr105/src/com/r_bg/stax/c14n/AttributeHandleExclusive.java

Author: mullan
Date: Thu Sep 13 14:29:23 2007
New Revision: 575466

URL: http://svn.apache.org/viewvc?rev=575466&view=rev
Log:
Fix bug in exclusive c14n. It was incorrectly emitting default namespace prefix 
if element had prefix itself. 

Modified:
    xml/security/branches/stax_jsr105/src/com/r_bg/stax/c14n/AttributeHandleExclusive.java

Modified: xml/security/branches/stax_jsr105/src/com/r_bg/stax/c14n/AttributeHandleExclusive.java
URL: http://svn.apache.org/viewvc/xml/security/branches/stax_jsr105/src/com/r_bg/stax/c14n/AttributeHandleExclusive.java?rev=575466&r1=575465&r2=575466&view=diff
==============================================================================
--- xml/security/branches/stax_jsr105/src/com/r_bg/stax/c14n/AttributeHandleExclusive.java (original)
+++ xml/security/branches/stax_jsr105/src/com/r_bg/stax/c14n/AttributeHandleExclusive.java Thu Sep 13 14:29:23 2007
@@ -23,7 +23,7 @@
 		for (int i=0;i<length;i++) {
 			args.add(new Integer(i));
 			String prefix=in.getAttributePrefix(i);
-			if (prefix!=null)
+			if (!prefix.isEmpty())
 				prefixes.add(prefix);
 		}		
 		prefixes.add(in.getPrefix()==null? "" : in.getPrefix());