You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@thrift.apache.org by "Henrique Mendonca (JIRA)" <ji...@apache.org> on 2012/11/29 08:14:58 UTC

[jira] [Commented] (THRIFT-1767) unions can't have required fields

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

Henrique Mendonca commented on THRIFT-1767:
-------------------------------------------

+1, thanks Jens!
... but shouldn't we throw a warning, at least, when compiling the idl?
                
> unions can't have required fields 
> ----------------------------------
>
>                 Key: THRIFT-1767
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1767
>             Project: Thrift
>          Issue Type: Bug
>          Components: Delphi - Compiler
>    Affects Versions: 0.8, 0.9
>            Reporter: Jens Geyer
>            Assignee: Jens Geyer
>             Fix For: 1.0
>
>         Attachments: THRIFT-1767_unions_cant_have_required_fields.patch
>
>
> Required fields within unions should be treaten as optional, the union construct would not make much sense otherwise. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira