Hi,
normally I would RFC first when doing changes to an XML processing API but this
one is really obvious: Without the change the IllegalArgumentException in
setProperty would be thrown in all cases.

I found it while trying to run a DeveloperWorks[0] demo application.

2005-10-02  Robert Schuster  <[EMAIL PROTECTED]>

    * gnu/xml/stream/XMLOutputFactoryImpl.java:
    (setProperty): Added 'else'.

cu
Robert

[0] - http://www-128.ibm.com/developerworks/xml/library/x-tipstx4/
Index: gnu/xml/stream/XMLOutputFactoryImpl.java
===================================================================
RCS file: /cvsroot/classpath/classpath/gnu/xml/stream/XMLOutputFactoryImpl.java,v
retrieving revision 1.1
diff -u -r1.1 XMLOutputFactoryImpl.java
--- gnu/xml/stream/XMLOutputFactoryImpl.java	4 Sep 2005 09:52:10 -0000	1.1
+++ gnu/xml/stream/XMLOutputFactoryImpl.java	2 Oct 2005 20:04:11 -0000
@@ -121,7 +121,8 @@
   {
     if (IS_PREFIX_DEFAULTING.equals(name))
       prefixDefaulting = ((Boolean) value).booleanValue();
-    throw new IllegalArgumentException(name);
+    else
+      throw new IllegalArgumentException(name);
   }
 
   public Object getProperty(String name)
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to