Author ncoghlan
Recipients amaury.forgeotdarc, barry, cvrebert, eli.bendersky, eric.snow, ethan.furman, ezio.melotti, giampaolo.rodola, gvanrossum, ncoghlan, pitrou, rhettinger
Date 2013-08-04.07:49:06
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1375602546.69.0.745119009172.issue18264@psf.upfronthosting.co.za>
In-reply-to
Content
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.
History
Date User Action Args
2013-08-04 07:49:06ncoghlansetrecipients: + ncoghlan, gvanrossum, barry, rhettinger, amaury.forgeotdarc, pitrou, giampaolo.rodola, ezio.melotti, eli.bendersky, cvrebert, ethan.furman, eric.snow
2013-08-04 07:49:06ncoghlansetmessageid: <1375602546.69.0.745119009172.issue18264@psf.upfronthosting.co.za>
2013-08-04 07:49:06ncoghlanlinkissue18264 messages
2013-08-04 07:49:06ncoghlancreate