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

David Reiss edited comment on THRIFT-72 at 3/24/09 1:37 PM:
------------------------------------------------------------

I have no problem with the basic design, as-is.  Some of the other languages 
already have similar features, and it is tough to require contributors to 
implement their feature in languages that they do not use.

      was (Author: dreiss):
    I have no problem with it, as-is.  Some of the other languages already have 
similar features, and it is tough to require contributors to implement their 
feature in languages that they do not use.
  
>  C++ structure constructor
> --------------------------
>
>                 Key: THRIFT-72
>                 URL: https://issues.apache.org/jira/browse/THRIFT-72
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Compiler (C++)
>         Environment: Tested on debian stable (etch)
>            Reporter: Eric Anderson
>            Priority: Minor
>             Fix For: 0.1
>
>         Attachments: structure-constructor.patch
>
>
> We're mostly using the C++ interface to thrift, and I wanted to be
> able to initialize structures more easily than a whole series of lines
> that set each of the parameters separately.  Attached is a patch that
> adds a constructor to the C++ objects that allow you to fully
> initialize an object in a single call.  I added two tests for this,
> one in the DebugProtoTest.cpp file, and one in the
> OptionalRequiredTest.cpp.  The patch is against the 20080411p1 release.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to