Re: [svgwg] Fix missing trailing wsp after path commands (#778)

Good initiative. I was waiting for more feedback from the committee to know whether I should submit a PR to fix/improve the grammar.

In general, I personally favor a concise grammar, more info here: 

https://github.com/w3c/svgwg/issues/751

In the specific case of the trailing whitespaces, here is what I advocate:

https://github.com/w3c/svgwg/issues/751#issuecomment-555163624

```
svg_path ::= wsp* (moveto (wsp* drawto_command)*)? wsp*
```

It is equivalent to your:

```
svg_path::= wsp* (moveto wsp* drawto_command_group*)? wsp*
drawto_command_group::=
    drawto_command
    | drawto_command wsp* drawto_command_group
```

But without the additional symbol `drawto_command_group` which makes the grammar three lines longer and harder for humans to parse. Less symbols = less cognitive load, in my opinion.

-- 
GitHub Notification of comment by dalboris
Please view or discuss this issue at https://github.com/w3c/svgwg/pull/778#issuecomment-593422367 using your GitHub account

Received on Monday, 2 March 2020 14:14:49 UTC