- From: Tom Shinnick via GitHub <sysbot+gh@w3.org>
- Date: Wed, 16 Aug 2017 21:37:40 +0000
- To: public-svg-issues@w3.org
@longsonr As I look now, I indeed misread which comma you were enabling. Not
```
"M0,0,L100,100"
^^
```
which everyone agrees isn't in the grammar, but rather you re-enabled
```
"M0,0,100,100"
^
```
where numbers for the implicit lineto commands follow directly after the moveto command. Commas between numbers are legal, and *only* legal *between* numbers.
And you [added a test](https://hg.mozilla.org/mozilla-central/rev/717711fc6d5d), `/layout/reftests/svg/path-08.svg` for this, so that a similar regression would not be unnoticed in the future.
I have to think I misread the fix partly because the fault was so unlikely. But the most amazing part of all this is that **this test did not exist originally**.
With a little looking around I can see the test was [copied into](https://svn.webkit.org/repository/webkit/trunk/LayoutTests/imported/mozilla/svg/path-08.svg) the Webkit project. That is certainly a good thing. But I'm not seeing the same or similar test under the [Blink project](https://chromium.googlesource.com/chromium/src/+/master/third_party/WebKit/LayoutTests) or the [web-platform-tests](http://web-platform-tests.org/) or [Github mirror](https://github.com/w3c/web-platform-tests). I can't guess after IE/Edge. Is it really true those projects don't have a test for this after 16 years? Is this area *really* so disorganized?
The meagerness of tests, the lack of rigor, for testing basic SVG things is astonishing to me. When, how do we make up for the oversight back then?
--
GitHub Notification of comment by tshinnic
Please view or discuss this issue at https://github.com/w3c/svgwg/issues/335#issuecomment-322906782 using your GitHub account
Received on Wednesday, 16 August 2017 21:37:47 UTC