xproc / 3.0-steps

Repository for change requests to the standard step library and for official extension steps
10 stars 7 forks source link

p:run: Suggestion for prose change #465

Closed xml-project closed 2 years ago

xml-project commented 4 years ago

Currently the specs reads:

The default readable port of the p:run step becomes the default readable port of the pipeline.

It took me about 5 times of reading and a phone call to understand what that sentence means.

Here is a suggestion to make it more understandable (to me):

The default readable port of the p:run step supplies the context item needed to evaluate the XPath expression provided by the select attribute in p:run-option. Additionally the default readable input port is connected to the p:run-input explicitly or implicitly marked as primary, if no explicit connection is provided.

I do not claim this to be perfect, but I think it does hint the direction.

ndw commented 4 years ago

I think it's an improvement. Grammatically, I might go for this:

The context item used to evaluate expressions on p:run-option comes from the default readable port of the p:run step. Additionally, if p:run-input implicitly or explicitly identifies a primary input port, the default readable port will be connected to it if no explicit connection is provided.

I think that says the same thing.

xml-project commented 4 years ago

Thanks, Norm. This is of course way better.

gimsieke commented 4 years ago

Consensus: Go with Norm’s wording

xatapult commented 2 years ago

@gimsieke Can this one be closed? It seems to be addressed.

ndw commented 2 years ago

Also, "yup".