[
https://issues.apache.org/jira/browse/THRIFT-72?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697240#action_12697240
]
Bryan Duxbury commented on THRIFT-72:
-------------------------------------
I see what your issue is here. THRIFT-236 was intended to have an effect on the
serialization order, but our solution to this problem inadvertently also had an
effect on things like constructors and service method parameter ordering. I
tend to agree that this change is undesired and we should probably fix it.
> 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
> Assignee: Alexander Shigin
> Priority: Minor
> Fix For: 0.1
>
> Attachments:
> 0001-THRIFT-72.-cpp-Add-a-struct-constructor-that-takes.patch,
> structure-constructor.patch, thrift-cpp-constructor-v2.patch,
> thrift-cpp-constructor-v3.patch, thrift-cpp-constructor-v4.patch,
> thrift-cpp-constructor-v5.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.
