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 explicit connections for secondary output ports #86

Merged
merged 1 commit into from Oct 15, 2018

Conversation

Projects
None yet
2 participants
@ndw
Contributor

ndw commented Oct 14, 2018

Fix #85

@ndw ndw requested a review from xml-project Oct 14, 2018

@xml-project

This comment has been minimized.

Contributor

xml-project commented Oct 15, 2018

@ndw Currently the specs still say that p:finally has no output port and I wasn't able to find an issue for the specs telling me, what to change. Could you please give me a hint what we decided about p:finally.

@ndw

This comment has been minimized.

Contributor

ndw commented Oct 15, 2018

The most recent thing I found was xproc/3.0-specification#128 which proposes arbitrary outputs. I recall some discussion of forbidding outputs, but it seems like a bad idea (to me, now). I'll make an issue to clarify this (in the specs repo).

@ndw ndw merged commit f4adbb4 into xproc:master Oct 15, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ndw ndw deleted the ndw:fix-try-catch branch Oct 15, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment