status-im/nim-json-serialization
2
0
Fork 0
mirror of https://github.com/status-im/nim-json-serialization.git synced 2025-02-16 12:07:35 +00:00
Code Issues Projects Releases Wiki Activity

Address review comments; Passing test suite

Browse Source
This commit is contained in:
Zahary Karadjov 2020-06-22 17:42:50 +03:00
parent 6e92113a06
commit 463a0cf1b7
No known key found for this signature in database
GPG Key ID: C8936F8A3073D609
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
json_serialization/reader.nim
View File

@ -52,7 +52,7 @@ func valueStr(err: ref IntOverflowError): string =
template tryFmt(expr: untyped): string =
try: expr
except CatchableError: ""
except CatchableError as err: err.msg
method formatMsg*(err: ref JsonReaderError, filename: string): string =
tryFmt: fmt"{filename}({err.line}, {err.col}) Error while reading json file"
@ -191,8 +191,8 @@ proc isNotNilCheck[T](x: ptr T not nil) {.compileTime.} = discard
template isCharArray[N](v: array[N, char]): bool = true
template isCharArray(v: auto): bool = false
proc readValue*(r: var JsonReader, value: var auto)
{.raises: [SerializationError, IOError, Defect].} =
proc readValue*[T](r: var JsonReader, value: var T)
{.raises: [SerializationError, IOError, Defect].} =
mixin readValue
let tok {.used.} = r.lexer.tok
@ -237,8 +237,11 @@ proc readValue*(r: var JsonReader, value: var auto)
elif value is enum:
case tok
of tkString:
# TODO: don't proprage the `parseEnum` exception
value.setParsed(r.lexer.strVal)
try:
value.setParsed(r.lexer.strVal)
except ValueError as err:
const typeName = typetraits.name(T)
r.raiseUnexpectedValue("Expected valid '" & typeName & "' value")
of tkInt:
# TODO: validate that the value is in range
value = type(value)(r.lexer.absIntVal)
@ -328,5 +331,5 @@ proc readValue*(r: var JsonReader, value: var auto)
r.skipToken tkCurlyRi
else:
const typeName = typetraits.name(value.type)
const typeName = typetraits.name(T)
{.error: "Failed to convert to JSON an unsupported type: " & typeName.}