Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

First-class string type in serialization specification #13

Closed
pgriess opened this issue Jul 23, 2010 · 7 comments
Closed

First-class string type in serialization specification #13

pgriess opened this issue Jul 23, 2010 · 7 comments
Labels

Comments

@pgriess
Copy link

pgriess commented Jul 23, 2010

Packing all strings as raw byte arrays makes it very difficult to figure out how to unpack them correctly. In particular, it is impossible to know what encoding was used when encoding the string as a sequence of bytes. To address this, it would be nice to have a first-class MSGPACK_OBJECT_STRING type with a mandatory encoding (say, UTF-8).

@aaronblohowiak
Copy link

any news on this?

@andrewschaaf
Copy link

Packing all strings as raw byte arrays makes it very easy to figure out how to unpack them correctly: as Buffers/byte[]s/....

Libraries could have

  • unpackRaw (doin' it rite)
  • unpackUnicode (attempting to autodetect encodings, starting with UTF-8)

instead of unpack

@tracker1
Copy link

@andrewschaaf The issue is more along the lines of dealing with cross-system messages. For example one system may have a native in-memory representation of strings as UTF-16, another may user UTF-8 ... since UTF-8 is usually the most effecient, it would make sense to have a string type that is always UTF-8 encoded without a BOM.

@tracker1
Copy link

For that matter, you could just put the UTF-8 encoded Byte Order Marker (BOM) at the beginning of your raw data, when reading out, you'll "know" that it's a UTF-8 string.

@cabo
Copy link

cabo commented Feb 20, 2013

The discussion of this issue is just exploding in #121

(And I'll plug http://tools.ietf.org/html/draft-bormann-apparea-bpack here, too.)

@cabo
Copy link

cabo commented Feb 24, 2013

Well, it seems we are continuing the technical discussion in #128 today.

@kuenishi
Copy link
Member

See the new spec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants