Bug 30718

Summary:	TransformerException in XSLURIResolver
Product:	classpath	Reporter:	Andrew Overholt <overholt>
Component:	classpath	Assignee:	chris burdess <dog>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aph, bug-classpath, tromey
Priority:	P3
Version:	0.92
Target Milestone:	0.94
Host:		Target:
Build:		Known to work:
Known to fail:		Last reconfirmed:	2007-02-07 08:56:55
Attachments:	Test case (boiled down from something I'm seeing with Mylar)

Description Andrew Overholt 2007-02-06 19:19:31 UTC

With the soon-to-be attached test case (javac [-1.5] *.java; java Main), I get no stack trace with Sun's VM when I do with classpath head (built from CVS using Eclipse) and cacao.  This prevents Mylar (an Eclipse plugin) from working correctly.

This is the stack trace:

(Note that the line numbers may be slightly off as I was adding debug statements)

javax.xml.transform.TransformerException: can't resolve URL: null
   at gnu.xml.transform.XSLURIResolver.resolveDOM(XSLURIResolver.java:187)
   at gnu.xml.transform.TransformerImpl.transform(TransformerImpl.java:123)
   at SaxRepositoriesWriter.writeRepositoriesToStream(SaxRepositoriesWriter.java:49)
   at TaskRepositoriesExternalizer.writeRepositoriesToXML(TaskRepositoriesExternalizer.java:27)
   at Main.main(Main.java:19)
java.io.IOException: can't resolve URL: null
   at SaxRepositoriesWriter.writeRepositoriesToStream(SaxRepositoriesWriter.java:54)
   at TaskRepositoriesExternalizer.writeRepositoriesToXML(TaskRepositoriesExternalizer.java:27)
   at Main.main(Main.java:19)

Comment 1 Andrew Overholt 2007-02-06 19:24:27 UTC

I can't seem to add an attachment (email sent to dberlin) so I've put the zip here:

http://overholt.ca/ResolverBug.zip

unzip ResolverBug.zip
cd ResolverBug
javac [-1.5] *.java
java Main

Comment 2 Andrew Overholt 2007-02-06 20:08:30 UTC

Created attachment 13015 [details]
Test case (boiled down from something I'm seeing with Mylar)

Comment 3 Tom Tromey 2007-02-06 20:33:07 UTC

Confirmed.

Comment 4 cvs-commit@developer.classpath.org 2007-02-07 10:09:28 UTC

Subject: Bug 30718

CVSROOT:	/cvsroot/classpath
Module name:	classpath
Changes by:	Chris Burdess <dog>	07/02/07 10:08:56

Modified files:
	.              : ChangeLog 
	gnu/xml/dom/ls : SAXEventSink.java 
	gnu/xml/transform: XSLURIResolver.java 

Log message:
	2007-02-07  Chris Burdess  <dog@gnu.org>
	
		Fixes PR 30718.
		* gnu/xml/dom/ls/SAXEventSink.java: Add public accessor/mutators.
		* gnu/xml/transform/XSLURIResolver.java: Add support for custom
		  SAXSources without a backing URL or stream.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/classpath/ChangeLog?cvsroot=classpath&r1=1.9090&r2=1.9091
http://cvs.savannah.gnu.org/viewcvs/classpath/gnu/xml/dom/ls/SAXEventSink.java?cvsroot=classpath&r1=1.7&r2=1.8
http://cvs.savannah.gnu.org/viewcvs/classpath/gnu/xml/transform/XSLURIResolver.java?cvsroot=classpath&r1=1.5&r2=1.6

Comment 5 chris burdess 2007-02-07 10:16:24 UTC

Test case now works as expected.

Comment 6 Andrew Haley 2007-02-07 14:05:37 UTC

Memo to Tom Tromey: can you check this in everywhere it's needed?

Comment 7 Tom Tromey 2007-02-07 18:22:57 UTC

Subject: Bug 30718

Author: tromey
Date: Wed Feb  7 18:22:26 2007
New Revision: 121694

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=121694
Log:
2007-02-07  Chris Burdess  <dog@gnu.org>

	Fixes PR 30718.
	* gnu/xml/dom/ls/SAXEventSink.java: Add public accessor/mutators.
	* gnu/xml/transform/XSLURIResolver.java: Add support for custom
	  SAXSources without a backing URL or stream.

	Fixes PR 27710.
	* gnu/xml/dom/DomDocumentBuilderFactory.java: Fall back to synchronous
	  LSParser if implementation does not support asynchronous.
	* gnu/xml/stream/XMLParser.java,
	  gnu/xml/stream/XIncludeFilter.java: Use custom code instead of
	  java.net.URL to resolve to an an absolute URI, to avoid nonexistent
	  protocol handler problems.

Modified:
    trunk/libjava/classpath/ChangeLog
    trunk/libjava/classpath/gnu/xml/dom/DomDocumentBuilderFactory.java
    trunk/libjava/classpath/gnu/xml/dom/ls/SAXEventSink.java
    trunk/libjava/classpath/gnu/xml/stream/XIncludeFilter.java
    trunk/libjava/classpath/gnu/xml/stream/XMLParser.java
    trunk/libjava/classpath/gnu/xml/transform/XSLURIResolver.java
    trunk/libjava/classpath/lib/gnu/xml/dom/DomDocumentBuilderFactory.class
    trunk/libjava/classpath/lib/gnu/xml/dom/ls/SAXEventSink.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XIncludeFilter.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$AnyContentModel.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$Attribute.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$AttributeDecl.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$ContentModel.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$ContentParticle.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$Doctype.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$ElementContentModel.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$EmptyContentModel.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$ExternalIds.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$Input.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser$MixedContentModel.class
    trunk/libjava/classpath/lib/gnu/xml/stream/XMLParser.class
    trunk/libjava/classpath/lib/gnu/xml/transform/XSLURIResolver.class
    trunk/libjava/gnu/xml/dom/ls/SAXEventSink.h
    trunk/libjava/gnu/xml/transform/XSLURIResolver.h

Comment 8 Tom Tromey 2007-02-07 18:29:45 UTC

Subject: Bug 30718

Author: tromey
Date: Wed Feb  7 18:29:30 2007
New Revision: 121696

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=121696
Log:
2007-02-07  Chris Burdess  <dog@gnu.org>

	Fixes PR 30718.
	* gnu/xml/dom/ls/SAXEventSink.java: Add public accessor/mutators.
	* gnu/xml/transform/XSLURIResolver.java: Add support for custom
	  SAXSources without a backing URL or stream.

	Fixes PR 27710.
	* gnu/xml/dom/DomDocumentBuilderFactory.java: Fall back to synchronous
	  LSParser if implementation does not support asynchronous.
	* gnu/xml/stream/XMLParser.java,
	  gnu/xml/stream/XIncludeFilter.java: Use custom code instead of
	  java.net.URL to resolve to an an absolute URI, to avoid nonexistent
	  protocol handler problems.

Modified:
    branches/redhat/gcc-4_1-branch-java-merge-20070117/libjava/classpath/ChangeLog
    branches/redhat/gcc-4_1-branch-java-merge-20070117/libjava/classpath/gnu/xml/dom/DomDocumentBuilderFactory.java
    branches/redhat/gcc-4_1-branch-java-merge-20070117/libjava/classpath/gnu/xml/dom/ls/SAXEventSink.java
    branches/redhat/gcc-4_1-branch-java-merge-20070117/libjava/classpath/gnu/xml/stream/XIncludeFilter.java
    branches/redhat/gcc-4_1-branch-java-merge-20070117/libjava/classpath/gnu/xml/stream/XMLParser.java
    branches/redhat/gcc-4_1-branch-java-merge-20070117/libjava/classpath/gnu/xml/transform/XSLURIResolver.java