Message194330
The two "isinstance" checks that bother me are the ones for int and float. However, given that the JSON serialiser *already* includes those explicit checks, I now think it makes sense to just do the minimal fix of coercing subclasses to the base type for both of those cases, especially since such a change should be able to be ported to simplejson as well.
Anything more elaborate (including making JSON explicitly enum aware, or else offering a new encoder that supports singledispatch) should be handled in a separate issue. |
|
Date |
User |
Action |
Args |
2013-08-04 07:49:06 | ncoghlan | set | recipients:
+ ncoghlan, gvanrossum, barry, rhettinger, amaury.forgeotdarc, pitrou, giampaolo.rodola, ezio.melotti, eli.bendersky, cvrebert, ethan.furman, eric.snow |
2013-08-04 07:49:06 | ncoghlan | set | messageid: <1375602546.69.0.745119009172.issue18264@psf.upfronthosting.co.za> |
2013-08-04 07:49:06 | ncoghlan | link | issue18264 messages |
2013-08-04 07:49:06 | ncoghlan | create | |
|