I'd prefer to go with Scott on this one, and make the Transformer et al work with the parser/reader/etc. to get the systemID. Theoretically the ContentHandler#setDocumentLocator call is optional, but most modern parsers should have one, and if they do, I'd expect to see xsltc look for it the same way xalan mode does.
In small device configurations I could see parsers that don't support this, in which case we'll barf; I'm not sure how best to address that. Overall I'd like to keep trax.sax as-is without code changes; if we want to add a trax.sax2 flavor that just adds that one line of setSystemID(), we could do that. But testing without the setSystemID is important, since a lot of users forget to call it, especially when they *know* they've already told the parser the actual URL. We've been somewhat successful at getting people to understand they had to set it when passing us an actual InputStream object, since in that case they can clearly see that they've never told us the URL. Make sense? ===== - Shane <eof aka="mailto:[EMAIL PROTECTED]"; .sig="Du sublime au ridicule il n'y a qu'un pas." /> __________________________________________________ Do You Yahoo!? Yahoo! Shopping - Mother's Day is May 12th! http://shopping.yahoo.com
