[THRIFT-3130] Remove the last vestiges of THRIFT_OVERLOAD_IF from THRIFT-1316

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.9.2
• Fix Version/s: 0.9.3
• Component/s: C++ - Library
• Assignee: James E. King III
• Reporter: James E. King III
• Created: 06/May/15 18:21
• Updated: 21/Jul/15 02:21
• Resolved: 07/May/15 13:17

Description

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.

Boost gained this mechanism internally in a release after boost-1.33 which is what was required when THRIFT-1316 was added to the codebase. With the current minimum requirement of 1.53, it is safe to remove THRIFT_OVERLOAD_IF. Tests confirm this (in TServerIntegrationTest).

Issue Links

• supercedes: THRIFT-1316 thrift: update server classes to accept