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

Konrad Grochowski commented on THRIFT-3130:
-------------------------------------------

oh, there is a confusion (at least for compiler) :D

it seems it in is resolved in boost API (they've included something like 
THRIFT_OVERLOAD_IF inside shared_ptr)

{code}
#if !defined( BOOST_SP_NO_SP_CONVERTIBLE )
    shared_ptr( shared_ptr<Y> const & r, typename 
boost::detail::sp_enable_if_convertible<Y,T>::type = boost::detail::sp_empty() )
#else
    shared_ptr( shared_ptr<Y> const & r )
#endif
{code}

BUT - why thrift needed OVERLOAD_IF? At least once I've encountered a 
compilation error due to lack of it... What changed?

> Remove the last vestiges of THRIFT_OVERLOAD_IF from THRIFT-1316
> ---------------------------------------------------------------
>
>                 Key: THRIFT-3130
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3130
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>    Affects Versions: 0.9.2
>            Reporter: James E. King, III
>            Assignee: James E. King, III
>            Priority: Minor
>
> In THRIFT-1316 some macros were added that have proven to be unnecessary.  
> They were partially eliminated in THRIFT-3083, and this traces the removal of 
> what was left.  In addition, TOutput was moved to its own class, and since 
> TThrift.cpp only contained TOutput code, it was renamed.



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

Reply via email to