You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2011/07/29 20:58:09 UTC
[jira] [Commented] (THRIFT-1243) TAsyncChannel callbacks
[ https://issues.apache.org/jira/browse/THRIFT-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13072976#comment-13072976 ]
Hudson commented on THRIFT-1243:
--------------------------------
Integrated in Thrift #195 (See [https://builds.apache.org/job/Thrift/195/])
THRIFT-1243 TAsyncChannel callbacks (use void instead of bool)
Patch: Alexandre Parenteau
roger : http://svn.apache.org/viewvc/?view=rev&rev=1151940
Files :
* /thrift/trunk/lib/cpp/src/async/TEvhttpClientChannel.cpp
* /thrift/trunk/lib/cpp/src/async/TAsyncChannel.cpp
* /thrift/trunk/lib/cpp/src/async/TEvhttpClientChannel.h
* /thrift/trunk/lib/cpp/src/async/TAsyncChannel.h
> 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
> Assignee: alexandre parenteau
> Priority: Minor
> Labels: patch
> Fix For: 0.8
>
> Attachments: THRIFT-1243.patch
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> [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