terwilliger42

Missing dependency due to default --max-bacon 2

4

Hi @thebjorn - thanks for all of your hard work on this project! Your tool is extremely helpful the work that I'm doing. I am so grateful. 👍 That said,...

Bools often corrupt when written using Compact

2

Looks like if you have a bool in your struct and you're writing using Compact format, it's very likely the resulting message will be corrupt. See attached testcase. This occurs...

Fixes #302 -- deprecation warnings when array.tostring() is called on Python 3.3+

4

Fixes #302; calls appropriate array.tobytes / array.tostring depending on Python version.

I'm getting deprecation warnings in compact.py when running my unittests on Python 3.5. Will submit a PR. ``` .../python3.5/site-packages/thriftpy/protocol/compact.py:419: DeprecationWarning: tostring() is deprecated. Use tobytes() instead. write_varint(self.trans, make_zig_zag(i32, 32)) .../python3.5/site-packages/thriftpy/protocol/compact.py:339:...

Sets are deserialized as lists by TBinaryProtocol, TCompactProtocol

1

When deserializing a thrift struct which contains a set, it is read into the object as a list, not as a set. For example: ``` import thriftpy from thriftpy.utils import...

Fix structs with > 254 fields (issue244)

3

- Add test which loads a struct with > 254 fields - Generated **init** now creates default args for only first 254 arguments to avoid compile SyntaxError exception. - SyntaxError...

struct can't contain more than 254 fields

4

When you add more than 254 fields to a struct, the attributes do not get initialized, the object cannot be serialized, etc. Traceback (most recent call last): File "thrift_test.py", line...