Remarks

1) In 5.8.1 Declaring atomic steps
[[

If p:log <http://www.w3.org/XML/XProc/docs/langspec.html#p.log> or
p:serialization<http://www.w3.org/XML/XProc/docs/langspec.html#p.serialization>elements
appear in the declaration of an atomic step, they will only be used
if the atomic step is directly evaluated by the processor. They are ignored
if the step appears in a pipeline.

]]

1.a) Please clarify the last sentence
1.b) Please explain what happen in this context (declaration of atomic
steps) when attributes amongst (@name, @psvi-required or @xpath-version)
are used ?
For example, not sure to understand what should happen, if a pipeline in
declared in p:library use  a  particular version, and the the declare-step
in  a  pipeline that reference that  library use  one, and that the
declare-step of this atomic step  use another one... ?


2) In 7.1.2 p:add-xml-base

Dandling "otherwise" ; please rewrite the "For other elements" for more
clarity

3) Please make consistent 7.2.7 p:www-form-urldecode and  7.1.17
p:parameters
One is output a sequence of c:param document, the other is outputing a
c:param-set
Since order is implementation defined, please use c:param-set for both.
Furthermore, I would add a constraint saying that no more than one c:param
with the same QName shoud be outputted in this c:param-set (it could
recalled in 5.1.2.2 The c:param-set element)

4) In 7.1.17

[[
Note

Since the parameters port is *not* primary, any explicit
p:with-param<http://www.w3.org/XML/XProc/docs/langspec.html#p.with-param>settings
must include a port attribute with value
p:with-param<http://www.w3.org/XML/XProc/docs/langspec.html#p.with-param>,
per the last paragraph of Section 5.7.4,
"p:with-param"<http://www.w3.org/XML/XProc/docs/langspec.html#p.with-param>
.

]]

Please replace "with value p:with-param" to "with value parameters"

-- 
Innovimax SARL
Consulting, Training & XML Development
9, impasse des Orteaux
75020 Paris
Tel : +33 9 52 475787
Fax : +33 1 4356 1746
http://www.innovimax.fr
RCS Paris 488.018.631
SARL au capital de 10.000 €