[
https://issues.apache.org/jira/browse/THRIFT-116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bryan Duxbury updated THRIFT-116:
---------------------------------
Attachment: thrift-116.patch
This patch removes isset fields for all object types, replacing them without
outright null checks. I've fixed the few tests that were broken by this (assist
by David), and everything seems to be working again.
> Isset fields for non-primitive types unnecessary
> ------------------------------------------------
>
> Key: THRIFT-116
> URL: https://issues.apache.org/jira/browse/THRIFT-116
> Project: Thrift
> Issue Type: Sub-task
> Components: Compiler (Java)
> Reporter: Bryan Duxbury
> Assignee: Bryan Duxbury
> Priority: Minor
> Attachments: thrift-116.patch
>
>
> For fields that aren't primitives, the Isset structure doesn't need to
> contain flags for those fields. This is because it is easy to detect whether
> it was set or not just by checking whether the field's value is null.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
