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

Konrad Grochowski commented on THRIFT-2000:
-------------------------------------------

not only calls: TNonblockingServer.h uses struct event

and in header which I needed to add to make it compile with 2.0.21 (0x020015) 
is such comment:

/** @file event2/event_struct.h

  Structures used by event.h.  Using these structures directly WILL harm
  forward compatibility: be careful.

  No field declared in this file should be used directly in user code.  Except
  for historical reasons, these fields would not be exposed at all.
 */

and those are the errors when trying to compile TNonblockingServer without 
event_compact.h:
thrift/lib/cpp/src/thrift/server/TNonblockingServer.cpp: In member function 
'void apache::thrift::server::TNonblockingServer::TConnection::setFlags(short 
int)':
thrift/lib/cpp/src/thrift/server/TNonblockingServer.cpp:815:44: error: 
'event_set' was not declared in this scope
thrift/lib/cpp/src/thrift/server/TNonblockingServer.cpp: In member function 
'void apache::thrift::server::TNonblockingIOThread::registerEvents()':
thrift/lib/cpp/src/thrift/server/TNonblockingServer.cpp:1341:22: error: 
'event_set' was not declared in this scope
thrift/lib/cpp/src/thrift/server/TNonblockingServer.cpp:1360:17: error: 
'event_set' was not declared in this scope

                
> TNonblocking server does not compile with libevent 2.0.21-stable
> ----------------------------------------------------------------
>
>                 Key: THRIFT-2000
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2000
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library
>    Affects Versions: 0.9
>         Environment: libevent 2.0.21
> gcc 4.7
>            Reporter: Konrad Grochowski
>         Attachments: libevent_2_0_21.patch
>
>
> problem lays in usage of deprecated libevent API, which is still available 
> but in different headers
> In patch there's ifdef based on libevent version, which will include missing 
> headers, but maybe some refactoring for future will be needed

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to