You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Chris Stylianou (Commented) (JIRA)" <ji...@apache.org> on 2012/02/21 10:20:35 UTC

[jira] [Commented] (THRIFT-1518) Generated C++ code only sends the first optional field in the write() function for a struct.

    [ https://issues.apache.org/jira/browse/THRIFT-1518?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13212479#comment-13212479 ] 

Chris Stylianou commented on THRIFT-1518:
-----------------------------------------

This seems to have resolved my issue thanks!
                
> Generated C++ code only sends the first optional field in the write() function for a struct.
> --------------------------------------------------------------------------------------------
>
>                 Key: THRIFT-1518
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1518
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Compiler
>    Affects Versions: 0.9
>            Reporter: Chris Stylianou
>         Attachments: THRIFT-1518_fix.patch
>
>
> The generated C++ code for a struct will only write the first optional field that is set before moving on to writing the required fields, instead it should be checking and writing all of the optional fields if they have been set. Looking at the generated code it is doing "if-else" tests for writing each optional field, this limits it to only writing the first optional field it comes across and should be easily fixed by replacing it with "if" checks instead.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira