Skip to content

Commit

Permalink
Disable @python.Py3EnableCppAdapter for typedef resolving to primitiv…
Browse files Browse the repository at this point in the history
…e and structs

Summary: Initial use cases seems like mostly containers, binary, and string. Let's restrict it.

Reviewed By: createdbysk

Differential Revision: D66511516

fbshipit-source-id: a10f50de4b23b779948078c048c397449993f951
  • Loading branch information
thedavekwon authored and facebook-github-bot committed Dec 9, 2024
1 parent 17fc918 commit b174a7c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions thrift/annotation/python.thrift
Original file line number Diff line number Diff line change
Expand Up @@ -97,5 +97,6 @@ struct UseCAPI {
}

/// Enables C++ Adapter for thrift-py3. It treats C++ Adapter on typedef as cpp.Type.
/// It is only available for typedefs that resolve to binary, string, and container type.
@scope.Typedef
struct Py3EnableCppAdapter {}

0 comments on commit b174a7c

Please sign in to comment.