public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Patch: FYI: fix TransformerImpl buglet
@ 2006-10-10 19:27 Tom Tromey
  0 siblings, 0 replies; only message in thread
From: Tom Tromey @ 2006-10-10 19:27 UTC (permalink / raw)
  To: GNU Classpath Patches; +Cc: Java Patch List

I'm checking this in to Classpath, gcc svn head, and the RH 4.1
branch.

This fixes PR 29362, a TransformerImpl buglet pointed out by the
Eclipse CDT.  I'm not completely sure that this is the best fix, but
we did test it against the CDT and it removed the particular failure
we were seeing.

FYI there's a self-contained test case in the PR.

Tom

2006-10-10  Tom Tromey  <tromey@redhat.com>

	PR classpath/29362:
	* gnu/xml/transform/TransformerImpl.java (transform): Only strip if
	there is a stylesheet.

Index: gnu/xml/transform/TransformerImpl.java
===================================================================
RCS file: /cvsroot/classpath/classpath/gnu/xml/transform/TransformerImpl.java,v
retrieving revision 1.15
diff -u -r1.15 TransformerImpl.java
--- gnu/xml/transform/TransformerImpl.java	7 Jun 2006 08:04:49 -0000	1.15
+++ gnu/xml/transform/TransformerImpl.java	10 Oct 2006 19:23:31 -0000
@@ -327,7 +327,8 @@
             domDoc.setCheckWellformedness(false);
           }
         parent.normalize();
-        strip(stylesheet, parent);
+        if (stylesheet != null)
+          strip(stylesheet, parent);
         Document resultDoc = (parent instanceof Document) ?
           (Document) parent :
           parent.getOwnerDocument();

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2006-10-10 19:27 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-10-10 19:27 Patch: FYI: fix TransformerImpl buglet Tom Tromey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).