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

James E. King, III commented on THRIFT-3336:
--------------------------------------------

Given the addition of compiler generated streaming out operators is an 
incompatible change, would it make more sense to make this a compile time 
option with the default on?  This would allow folks to easily turn off the 
emission of the streaming operator so that they can continue to use their own.  
Thoughts?

> Thrift generated streaming operators added in 0.9.2 cannot be overridden
> ------------------------------------------------------------------------
>
>                 Key: THRIFT-3336
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3336
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Compiler
>    Affects Versions: 0.9.2, 0.9.3
>            Reporter: James E. King, III
>            Assignee: James E. King, III
>
> I work on one project that has been using thrift-0.8.0 and I am updating it 
> to 0.9.3.  The project defines streaming operators for "operator <<" on a 
> variety of thrift-generated types.
> The fix added in THRIFT-2067 provides similar but not identical functionality 
> however it was done in a way that cannot be overridden, and this causes 
> projects upgrading from versions prior to 0.9.2 with their own streaming 
> operators to fail to compile.
> One way to make the streaming operator virtual is to follow the example 
> provided in 
> http://stackoverflow.com/questions/5189614/operator-and-inheritance



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to