feature request: toJSON() about node-msgpack HOT 3 CLOSED

toJSON is normally a function used for returning JSON serializable objects from an object that doesn't allow enumeration, or for when you want JSON to only serialize a portion of the object.
For example:

function A() {
  this.t = 4;
  this.prop = "test";
}
A.prototype.toJSON = function() {
  return {prop: this.prop}
}

var a = new A;

If a is ever stringifyed with JSON, you will end up with {"prop": "test"} instead of {"t": 4, "prop": "test"}.

tl;dr You might want to change this request to use a different function name.

from node-msgpack.

Can you help clarify? If I have an array:

var myArray = [ "one", "two" ];

and I pack that with msgpack.pack:

var binaryData = msgpack.pack(myArray);

You want a function that let's you say:

var jsonString = msgpack.unpack(binaryData, 'JSON');

I tend to agree with download13, we should not call it msgpack.toJSON(). I think making something like this an option to unpack is more desirable.

Is this what you mean?

from node-msgpack.

closing because of #50

from node-msgpack.