[ https://issues.apache.org/jira/browse/THRIFT-1305?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dave Watson updated THRIFT-1305: -------------------------------- Attachment: 0024-thrift-make-TConnection-a-private-inner-class-of-TNo.patch > thrift: make TConnection a private inner class of > --------------------------------------------------- > > Key: THRIFT-1305 > URL: https://issues.apache.org/jira/browse/THRIFT-1305 > Project: Thrift > Issue Type: Improvement > Components: C++ - Library > Reporter: Dave Watson > Priority: Minor > Attachments: > 0024-thrift-make-TConnection-a-private-inner-class-of-TNo.patch > > > From 1f203754d8e9eb3e509cd874f6e5824eeaf5aa34 Mon Sep 17 00:00:00 2001 > From: Adam Simpkins <simpk...@fb.com> > Date: Mon, 19 Apr 2010 19:09:13 +0000 > Subject: [PATCH 24/33] thrift: make TConnection a private inner class of > TNonblockingServer > Summary: > The immediate need for this is so that TConnection can access some > protected methods of TNonblockingServer. However, it > seems a generally good thing to eliminate TConnection from the publicly > exposed API. > Test Plan: > Ran the tests from [fb unittest]. > Revert Plan: > OK > Conflicts: > lib/cpp/src/server/TNonblockingServer.cpp > lib/cpp/src/server/TNonblockingServer.h > --- > lib/cpp/src/server/TNonblockingServer.cpp | 301 > +++++++++++++++++++++++++++-- > lib/cpp/src/server/TNonblockingServer.h | 275 +-------------------------- > 2 files changed, 291 insertions(+), 285 deletions(-) -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira