diffix / reference

Reference implementation for the Open Diffix anonymization mechanism.
https://www.open-diffix.org
Other
3 stars 0 forks source link

Fix parsing of Floats like 1.101 and 1.lots-of-digits #383

Closed pdobacz closed 2 years ago

pdobacz commented 2 years ago

I picked it up when using WHERE x < 40.63 for debugging - it was parsing as 46.3.

I've made an initial implementation fixing the one already there with some more code, but it started to become a considerable chunk to handle all the cases. So I redid the search and found the numberLiteral parser in FParsec - I hope this works.

To be sure we're covered, I added some extra test cases. Possibly repetitive since we're using FParsec now, but I think worth to have in case we stop doing so.