[
https://issues.apache.org/jira/browse/THRIFT-3968?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15693554#comment-15693554
]
Chandler May commented on THRIFT-3968:
--------------------------------------
Hi [~simonsouth], I'm sorry for such a long delay. This looks good to me.
Also, it seems empty binary fields are read the same way. Should we create an
analogous issue and patch for it?
> Deserializing empty string
> --------------------------
>
> Key: THRIFT-3968
> URL: https://issues.apache.org/jira/browse/THRIFT-3968
> Project: Thrift
> Issue Type: Bug
> Components: C glib - Compiler, C glib - Library
> Affects Versions: 0.9.3, 1.0
> Reporter: Matej Kupljen
> Assignee: Simon South
> Attachments: 0-thrift-3968-add-test-cases.patch,
> 1-thrift-3968-deserialize-empty-string.patch
>
>
> If we use a string inside struct definition and set the value to "" (empty
> string, not NULL) when de-serializing it we get set it NULL and and the same
> time the __isset field for this value is set to TRUE which is think is not
> correct.
> Either we fix the library to return an empty string, or fix the compiler to
> set the __isset filed to FALSE.
> What would be the best way to solve this?
> Please, let me know if you need any more information.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
