Informative parse errors about terraformer-wkt-parser HOT 8 CLOSED

we might be able to pass back the portion of the line that the parser fails on, i can look into that.

we're using JISON under the hood with a BNF grammar to generate the actual parser.

from terraformer-wkt-parser.

thanks for considering this :)

from terraformer-wkt-parser.

looks like it should be working, but i'll try to find some time tomorrow to investigate why it isn't:

https://github.com/Esri/terraformer-wkt-parser/blob/master/src/module-source.js#L111

from terraformer-wkt-parser.

looks like the error wasn't being properly thrown and the second argument never got printed.

this is what it should look like:

Error: Unable to parse: Error: Parse error on line 1:
...G (30 10, 10 30, 40 a)
-----------------------^
Expecting 'DOUBLE_TOK', got 'INVALID'

updating to make that happen.

from terraformer-wkt-parser.

great! that looks quite useful

from terraformer-wkt-parser.

it is, and indeed it is as useful as it was meant to be :)

just finishing up getting it all built and packaged and should hopefully have a release out shortly. i'll close the ticket when that happens.

from terraformer-wkt-parser.

👍

from terraformer-wkt-parser.

done and live

from terraformer-wkt-parser.