Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add UnparsedTextURIResolver in XSLT.java (need to have get/setUnparsedTe... #32

Closed
wants to merge 1 commit into from

Conversation

@innovimax
Copy link
Contributor

@innovimax innovimax commented Mar 2, 2012

...xtURIResolver on com.xmlcalabash.core.XProcRuntime)

…dTextURIResolver on com.xmlcalabash.core.XProcRuntime
@innovimax
Copy link
Contributor Author

@innovimax innovimax commented Mar 2, 2012

@ndw

This comment has been minimized.

Copy link

@ndw ndw commented on 80a250d Mar 15, 2012

There must be more to this patch. XProcRuntime doesn't have a getUnparsedTextURIResolver() method.

This comment has been minimized.

Copy link
Owner Author

@innovimax innovimax replied Mar 15, 2012

oups... indeed

@innovimax
Copy link
Contributor Author

@innovimax innovimax commented Mar 26, 2012

Well in fact I didn't forgot : that written in the title " (need to have get/setUnparsedTextURIResolver on com.xmlcalabash.core.XProcRuntime)"

@ndw
Copy link
Owner

@ndw ndw commented Apr 4, 2012

Well, uh, have you implemented those bits too? Would you like to make them part of the patch? :-)

@innovimax
Copy link
Contributor Author

@innovimax innovimax commented Apr 5, 2012

There are just dummy getter and setter of a private object, aren't they?

@ndw
Copy link
Owner

@ndw ndw commented Jan 29, 2013

Nudge: I sent you a complete patch in email to test. Please let me know if it works.

@ndw
Copy link
Owner

@ndw ndw commented Jan 30, 2013

Impatiently assuming my patch works. :-)

Let me know if I'm wrong!

@ndw ndw closed this Jan 30, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.