Re: [patch] NPE bug in gnu.xml.aelfred2.JAXPFactory

Arnaud Vandyck <[email protected]> Tue, 20 Apr 2004 10:32:56 +0200
Newsgroups gmane.comp.java.classpath.extensions.discuss
Message-ID <[email protected]>
Seems good to me, patch applyed.

Thanks Maarten!

Maarten Coene <[email protected]> writes:

> Hi,
>
> the gnu.xml.aelfred2.JAXPFactory.getFeature(String name) method throws a
> NullPointerException if you check for a feature that hasn't been set at
> the factory level.
>
> Here is a patch that fixes this.
>
> regards,
> Maarten
>
> Index: JAXPFactory.java
> ===================================================================
> RCS file: /cvsroot/classpathx/jaxp/src/gnu/xml/aelfred2/JAXPFactory.java,v
> retrieving revision 1.2
> diff -u -r1.2 JAXPFactory.java
> --- JAXPFactory.java    14 Apr 2004 15:40:02 -0000    1.2
> +++ JAXPFactory.java    19 Apr 2004 14:35:39 -0000
> @@ -117,7 +117,7 @@
>      {
>      Boolean    value = (Boolean) flags.get (name);
>     -    if (value == null)
> +    if (value != null)
>          return value.booleanValue ();
>      else
>          try {
>
>
>
> _______________________________________________
> Classpathx-discuss mailing list
> [email protected]
> http://mail.gnu.org/mailman/listinfo/classpathx-discuss
>
>

-- 
Arnaud Vandyck

I'm personally quite happy with one stable release every two years, and
am of the opinion that trying to release more will mean we'll have to
rename the distro from "stable" to "wobbly".
		-- Scott James Remnant on debian-devel