Add a toJSON about long.js HOT 3 CLOSED

Seems that it currently returns the long's internal state, which are the low 32 and the high 32 bits which in turn make up its 64 bit value. The third field indicates whether the long is to be interpreted as unsigned or not (signed). Unfortunately, a string conversion would lose the signed information.

Imho this should be handled by the developer, may it be adding the desired toJSON implementation to Long.prototype or something else which transparently calls toString.

from long.js.

I'm handling it myself as you suggested, but I did expect it to return the value as a string. One example of this is how Date handles it:

> new Date().toJSON()
'2016-06-28T23:59:07.147Z'

from long.js.

Seems like something more useful would be a JSON decoder which can deconstruct the object representation. You'd need to hack the global JSON.parse implementation a bit so it should be opt-in. Something like:

Long.parseJSONLongs = () => {
    const oldJSONparse = JSON.parse;
    JSON.parse = (json, transform=(x => x)) => {
	return oldJSONparse(json, (x) => {
		if ('low' in x && 'high' in x && 'unsigned' in x) {
			return transform(new Long(x.low, x.high, x.unsigned));
		} else {
			return transform(x);
		}
	});
    };
}

from long.js.