status-im / nim-toml-serialization

Flexible TOML serialization [not] relying on run-time type information.
Apache License 2.0
36 stars 7 forks source link

Cannot catch `OverflowError` #58

Closed arnetheduck closed 1 year ago

arnetheduck commented 1 year ago

OverflowError is a Defect (OverflowDefect in nim 1.4+) - trying to catch it will not work as expected: https://github.com/status-im/nim-toml-serialization/blob/f73f8a9e3da19b5297663fe40229d118af0999ed/toml_serialization/reader.nim#L93, thus the code will need refactoring