[css-shapes-2] Some arc and curve command parameters should not be optional

[cdoublev]

Following this resolution and the accepted PR, the syntax of <arc-command> is arc [[<by-to> <coordinate-pair>] || [of <length-percentage>{1,2}] || <arc-sweep>? || <arc-size>? || [rotate <angle>]?].

<by-to> <coordinate-pair> and of <length-percentage>{1,2} are now optional but this does not seem to be the intent of the resolution: "switch to using, allow reordering grammar".

The same problem applies to:

• <curve-command> = curve [[<by-to> <coordinate-pair>] || [using <coordinate-pair>{1,2}]]
• <smooth-command> = smooth [[<by-to> <coordinate-pair>] || [using <coordinate-pair>]?].

If this is not intentional, I suggest:

<arc-command> = arc [
     <by-to> <coordinate-pair>
  && of <length-percentage>{1,2}
  && <arc-sweep>?
  && <arc-size>?
  && [rotate <angle>]?
]
<curve-command> = curve [<by-to> <coordinate-pair> && using <coordinate-pair>{1,2}]
<smooth-command> = smooth [<by-to> <coordinate-pair> && [using <coordinate-pair>]?]

[noamr]

Seems right, I can prepare a PR (I don't think we need a new resolution)

[noamr]

This is fixed

[cdoublev]

Thanks btw.

Link to the related change: ddcbffd#diff-8456ef5bba8c9386e8c499974f617d442407c3404ff7bb0b19fee3d1f9671ef6R194