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

Daniel Haensse commented on THRIFT-4867:
----------------------------------------

Attached the struct less operator patch with patching the tests to get it 
compiled. 

Where do you want the unit tests to be added best (if required)?

> C++ operator< implementation is not generated on structs
> --------------------------------------------------------
>
>                 Key: THRIFT-4867
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4867
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Compiler
>    Affects Versions: 0.12.0
>         Environment: all
>            Reporter: Daniel Haensse
>            Priority: Minor
>         Attachments: 
> thrift-0.12.0_t_cpp_generator_struct_less_operator.patch, 
> thrift-0.12.0_t_cpp_generator_struct_less_operator_DebugProtoTest_extras.cpp.patch,
>  thrift-0.12.0_t_cpp_generator_struct_operator_less_ThriftTest_extras.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> When using struct in list, list and set, the operator< is required. 
> Unfortunately the c++ compiler even in the latest release does not generate 
> it. It would be very usefull to have the implementation generated, reducing 
> unnecessary manual coding. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to