Skip to content

Conversation

@umegaya
Copy link

@umegaya umegaya commented Dec 18, 2012

以前のpull requestではstream unpackの対応が完全でなかったのでいかのように修正しなおしました。

  • 複数のストリームをサポートするためにはグローバルな変数は使えないので、msgpack.stream_unpacker()でストリームの状態を保持するオブジェクトを作成、それを使ってunpackを行うようにし、msgpack.unpackはそのまま残すようにした。
  • テストの追加
    -- test/stream_unpack.htm
    -- test/codec.htm?stream=1

stream_unpackerが内部で利用するstream_decode_tokenはdecode()と処理が結構重複していますが、高速に動作させることを意図しているdecodeの実装を考えると共有は難しいと考えたため、重複させたままにしています

commit f57da25
Author: iyatomi takehiro [email protected]
Date: Sat Dec 15 03:39:56 2012 +0900

adjust stream unpack test to project's standard way

commit c389bd1
Author: iyatomi takehiro [email protected]
Date: Sat Dec 15 03:04:35 2012 +0900

add stream unpacker and its test

commit f57da25
Author: iyatomi takehiro <[email protected]>
Date:   Sat Dec 15 03:39:56 2012 +0900

    adjust stream unpack test to project's standard way

commit c389bd1
Author: iyatomi takehiro <[email protected]>
Date:   Sat Dec 15 03:04:35 2012 +0900

    add stream unpacker and its test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant