cedric
10/10/2018, 7:12 PMShawn
10/10/2018, 7:18 PMKlaxonException
with the specific parsing errorcom.beust.klaxon.KlaxonException: Expected "}", a value or ",", not 'EOF' at line 1
arve
10/10/2018, 7:18 PMcedric
10/10/2018, 7:19 PMarve
10/10/2018, 7:20 PMdata class Thing(val foo: String)
assert(Klaxon().parse<Thing>("]") == null)
parse
is T?
insted of T
and b) create a test for my helper methods' null handling 🙂keturn
10/10/2018, 7:27 PMkarelpeeters
10/10/2018, 7:34 PMnull
, since at some point the parse
functions call parseFromJsonObject
which calls fromJsonObject
, that returns Any
and the result is cast to T?
.arve
10/10/2018, 7:42 PMT
.. Would a PR for this be welcome? @cedriccedric
10/10/2018, 7:43 PM* @return a JsonObject or JsonArray
Really wish this didn’t have to be a comment 🙂Either<JsonObject, JsonArray>
insteadkarelpeeters
10/10/2018, 8:47 PMcedric
10/10/2018, 8:48 PMkarelpeeters
10/10/2018, 8:50 PMcedric
10/10/2018, 8:51 PMparse(): Any?
JsonObjectOrArray
I guesskarelpeeters
10/10/2018, 8:53 PMcedric
10/10/2018, 8:54 PMJsonObjectOrArray
? Open to suggestionsBaseJson
?Shawn
10/10/2018, 8:54 PMBaSON
, surely 😛karelpeeters
10/10/2018, 8:59 PMJsonBase
then for consistency.cedric
10/10/2018, 8:59 PMkarelpeeters
10/10/2018, 8:59 PMcedric
10/10/2018, 8:59 PMparse()
and parseArray()
karelpeeters
10/10/2018, 9:00 PMcedric
10/10/2018, 9:00 PMAny
JsonObject
and JsonArray
already extend JSonBase
, so we’re done here 🙂karelpeeters
10/10/2018, 9:02 PMsealed interface
cedric
10/10/2018, 9:03 PM