Skip to content

Squashed commit of the following: #6

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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 iyatomi@gmail.com
Date: Sat Dec 15 03:39:56 2012 +0900

adjust stream unpack test to project's standard way

commit c389bd1
Author: iyatomi takehiro iyatomi@gmail.com
Date: Sat Dec 15 03:04:35 2012 +0900

add stream unpacker and its test

commit f57da25
Author: iyatomi takehiro <iyatomi@gmail.com>
Date:   Sat Dec 15 03:39:56 2012 +0900

    adjust stream unpack test to project's standard way

commit c389bd1
Author: iyatomi takehiro <iyatomi@gmail.com>
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