- From: Boris Dalstein via GitHub <sysbot+gh@w3.org>
- Date: Tue, 03 Mar 2020 18:01:34 +0000
- To: public-svg-issues@w3.org
> It took me three iterations banging my head on a wall to start having a reasonable understanding of the path grammar. Same here :) Thx for the amend to your PR! I do find this 1-liner version the most readable despite the nested parentheses, but it's obviously very subjective. Note that in your 2-line suggestion above, the `+` in `(drawto_command wsp*)+` is redundant with the `*` in `drawto_command_group*` (in other words, `(A+)*` and `A*` are equivalent), so you could simplify the suggestion to: ``` svg_path::= wsp* (moveto wsp* drawto_command_group*)? drawto_command_group::= drawto_command wsp* ``` (in which case the name `drawto_command_group` should obviously be changed, e.g. to `command_wsp`, but then since it's so simple and only used once... well, we might as well inline it and avoid finding a good name ;) ) -- GitHub Notification of comment by dalboris Please view or discuss this issue at https://github.com/w3c/svgwg/pull/778#issuecomment-594085821 using your GitHub account
Received on Tuesday, 3 March 2020 18:01:36 UTC