TAsyncChannel callbacks
-----------------------

                 Key: THRIFT-1243
                 URL: https://issues.apache.org/jira/browse/THRIFT-1243
             Project: Thrift
          Issue Type: Improvement
          Components: C++ - Library
    Affects Versions: 0.7
         Environment: Visual C++ 2010
            Reporter: alexandre parenteau
            Priority: Minor
             Fix For: 0.8


[Context: This patch is part of a larger patch to use thriftnb on 
Windows/VisualC++. See 
https://github.com/aubonbeurre/thrift/blob/alex-0.7.0/README.non.blocking.Windows
 for more details.]

When compiling using Visual C++ 2010, the compiler chokes on casting some 
callbacks bool(*)() to void(*)().

Although probably valid and supported by gcc, this is further complicated by 
the fact those casting seem unnecessary: for each of the callbacks returning 
bool in TAsyncChannel.h:

1. the returned value is never checked
2. they always return true

Attached is a trivial patch based on 0.7.0, tested on Ubuntu 11.04 and Visual 
C++ 2010.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to