[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] json: Add support for encoding structs
From: |
Stefan Monnier |
Subject: |
Re: [PATCH] json: Add support for encoding structs |
Date: |
Mon, 24 Apr 2017 17:43:21 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
> I also want to ask if the keywords could be less JSON-specific, since
> other serializers could be used. So you'd use keywords like
> :serialize-field, :serialize-ignore, :serialize-to-type instead of the
> ones you proposed.
Indeed. Other thing: please try and think about the impact those field
might have on the round-tripping of data through JSON (i.e. how to make
it so that decoding the JSON output back to Elisp gives equivalent data).
> Finally, maybe we could also consider using the existing facility
> :print-function (the manual says that's currently unused).
I think this :print-function is now obsolete since it was meant to
hook into `princ`, but that can now be done in cl-print.
> But I don't know the full implications of that, so maybe it's
> a bad suggestion.
I don't think :print-function and cl-print can be of much use for
encoding to JSON.
Stefan
- [PATCH] json: Add support for encoding structs, Vibhav Pant, 2017/04/23
- Re: [PATCH] json: Add support for encoding structs, Ted Zlatanov, 2017/04/24
- Re: [PATCH] json: Add support for encoding structs, Vibhav Pant, 2017/04/24
- Re: [PATCH] json: Add support for encoding structs, Ted Zlatanov, 2017/04/24
- Re: [PATCH] json: Add support for encoding structs, Ted Zlatanov, 2017/04/24
- Re: [PATCH] json: Add support for encoding structs,
Stefan Monnier <=
- Re: [PATCH] json: Add support for encoding structs, Ted Zlatanov, 2017/04/25
- Re: [PATCH] json: Add support for encoding structs, Vibhav Pant, 2017/04/25
- Re: [PATCH] json: Add support for encoding structs, Ted Zlatanov, 2017/04/25
- Re: [PATCH] json: Add support for encoding structs, Stefan Monnier, 2017/04/26
- Re: [PATCH] json: Add support for encoding structs, Vibhav Pant, 2017/04/27
- Re: [PATCH] json: Add support for encoding structs, Stefan Monnier, 2017/04/27
- Re: [PATCH] json: Add support for encoding structs, Vibhav Pant, 2017/04/25
- Re: [PATCH] json: Add support for encoding structs, Davis Herring, 2017/04/24
- Re: [PATCH] json: Add support for encoding structs, Vibhav Pant, 2017/04/25
- Re: [PATCH] json: Add support for encoding structs, Clément Pit-Claudel, 2017/04/24