[ovs-dev] [PATCH] JSON serialization via Python's json lib
Ben Pfaff
blp at ovn.org
Tue Jul 26 16:19:13 UTC 2016
On Mon, Jul 25, 2016 at 07:17:11PM -0500, Terry Wilson wrote:
> There is no particularly good reason to use our own Python JSON
> serialization implementation when serialization can be done faster
> with Python's built-in JSON library.
>
> A few tests were changed due to Python's default JSON library
> returning slightly more precise floating point numbers.
>
> Signed-off-by: Terry Wilson <twilson at redhat.com>
Applied, thanks.
This patch raises the question of whether OVS should be in the business
of testing Python's built-in libraries. That is, maybe the Python
versions of the JSON tests should go away entirely? I don't know.
Along the same lines, maybe the code that uses the OVS json python
module, at least for serialization, should be changed to just use the
Python built-in json module directly.
None of this is urgent or important.
More information about the dev
mailing list