Re: [css-houdini-drafts] [css-properties-values-api] Fix syntax string parsing to not treat whitespace between components the same as pipes. (#894)

Let's rethink step 5 as a whole. Drop the "repeatedly consume" thing and instead do:

Step 5:
 * Consume a syntax component from stream. If failure was returned, return failure; otherwise, append the returned value to descriptor. [Note: Consume a syntax component consumes leading whitespace, which is relevant for repeats of this step].
 * Consume as much whitespace as possible from stream.
 * Consume the next input code point from stream:
   * EOF: Return descriptor. [Note: I don't think we still need the descriptor size check, since we'll now exit early if "Consume a syntax component" fails?]
   * U+007C VERTICAL LINE (|):  Repeat this step.
   * Anything else: Return failure.

WDYT?

-- 
GitHub Notification of comment by andruud
Please view or discuss this issue at https://github.com/w3c/css-houdini-drafts/pull/894#issuecomment-497615397 using your GitHub account

Received on Friday, 31 May 2019 07:59:10 UTC