[
https://issues.apache.org/jira/browse/THRIFT-1287?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bryan Duxbury closed THRIFT-1287.
---------------------------------
Resolution: Fixed
Fix Version/s: 0.8
Assignee: Dave Watson
I just committed this.
> thrift: start refactoring some of the C++ processor
> -----------------------------------------------------
>
> Key: THRIFT-1287
> URL: https://issues.apache.org/jira/browse/THRIFT-1287
> Project: Thrift
> Issue Type: Improvement
> Components: C++ - Library
> Reporter: Dave Watson
> Assignee: Dave Watson
> Priority: Minor
> Fix For: 0.8
>
> Attachments:
> 0014-thrift-start-refactoring-some-of-the-C-processor-cod.patch
>
>
> From 527b9dc312e3dc45a18e15b50c0daf7dab52bfb5 Mon Sep 17 00:00:00 2001
> From: Adam Simpkins <[email protected]>
> Date: Sat, 3 Apr 2010 02:49:48 +0000
> Subject: [PATCH 14/33] thrift: start refactoring some of the C++ processor
> code generation
> Summary:
> This adds a ProcessorGenerator class to store all of the local variables
> used by generate_service_processor(). This way we can break up that
> giant function into some smaller pieces.
> The code is still pretty ugly, but it's a start.
> Test Plan:
> Built [two fb projects], and some additional thrift test files, and made
> sure the generated files were identical both before and after these
> changes.
> Revert Plan:
> OK
> Conflicts:
> compiler/cpp/src/generate/t_cpp_generator.cc
> ---
> compiler/cpp/src/generate/t_cpp_generator.cc | 346
> ++++++++++++++++----------
> 1 files changed, 216 insertions(+), 130 deletions(-)
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
