facebookarchive / swift

An annotation-based Java library for creating Thrift serializable types and services.
Apache License 2.0
900 stars 297 forks source link

Add support for Java 8 Optional #261

Closed vicmosin closed 6 years ago

vicmosin commented 9 years ago

Could you please give some feedback regarding supporting this issue? Especially when Thrift/Swift doesn't support any null values and the official suggestions (Use a wrapper struct or a marker value instead) are not perfect solution.

dain commented 9 years ago

We should just support Optional directly in Swift.

xiemeilong commented 8 years ago

+1

m-hogue commented 7 years ago

+1