[
https://issues.apache.org/jira/browse/THRIFT-72?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12703491#action_12703491
]
Alexander Shigin commented on THRIFT-72:
----------------------------------------
The original Eric's patch
(http://publists.facebook.com/pipermail/thrift/2008-July/001071.html) doesn't
do any sorting.
It was my mail
(http://publists.facebook.com/pipermail/thrift/2008-July/001072.html) which
proposed field sorting. Mark Slee agreed with me
(http://mail-archives.apache.org/mod_mbox/incubator-thrift-dev/200807.mbox/%3c028001c8e259$c7307eb0$31101...@ieslee%3e).
> 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.
