| commit | 543caf6a5238300a8f7d523fdfb08b6dbb04dac1 | |
| author | Filip Francetic <ffrancet@meta.com> | |
| Thu, 30 Jan 2025 22:31:14 +0000 (30 14:31 -0800) | ||
| committer | Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com> | |
| Thu, 30 Jan 2025 22:31:14 +0000 (30 14:31 -0800) | ||
| tree | deca29a2d31190e5eebe4bc1a513b940b02ae83d | treesnapshot (tar.gz zip) |
| parent | 411280d5d40542a0a6fb29acb714e7291b7ad32a | commitdiff |
Add a capi conversion no-op method to thrift python structs/unions
Summary: There's a generated python capi method on thrift-py3 structs to initialize from thrift python types. It's a very simple fix to add a no-op method to the base thrift python StructOrUnion class to get these unit tests to start passing now. Since the method is meant to initialize a thrift python struct with a thrift python struct, all that we need to do is return the passed in object (after a basic instance check to make sure no python shenanigans are going on).
Reviewed By: ahilger
Differential Revision: D68867840
fbshipit-source-id: 817381af3586cf7586532c6f94af676cfc16b241
Summary: There's a generated python capi method on thrift-py3 structs to initialize from thrift python types. It's a very simple fix to add a no-op method to the base thrift python StructOrUnion class to get these unit tests to start passing now. Since the method is meant to initialize a thrift python struct with a thrift python struct, all that we need to do is return the passed in object (after a basic instance check to make sure no python shenanigans are going on).
Reviewed By: ahilger
Differential Revision: D68867840
fbshipit-source-id: 817381af3586cf7586532c6f94af676cfc16b241