Closed cage2 closed 4 years ago
On Tue, May 26, 2020 at 09:58:27AM -0700, Nicolas Hafner wrote:
Hi Shinmera!
@Shinmera commented on this pull request.
[...]
+(defgeneric maybe-parse-integer (object))
maybe-parse-integer
is somewhat confusing of a name. I'd call itensure-integer
instead.
I agree it is confusing! changed!
Bye! C.
Hi Shinmera!
I have added a few missing slots from class
status
.Moreover seems to me that
yason
already parses numeric field to lisp number as i am getting type errors like :0 is not a string
when:translate-with #'parse-integer
is used, to sort of 'fix' this, in the meanwhile, i have added this function:maybe-parse-integer
that try to do the right thing. ;-)Also updated and fixed the documentation.
Bye!