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

Hudson commented on THRIFT-1283:
--------------------------------

Integrated in Thrift #230 (See [https://builds.apache.org/job/Thrift/230/])
    THRIFT-1283. cpp: wrap t_cpp_generator::generate_process_function() to 80

Adjust the line width to 80 for generated CPP classes.

Patch: Adam Simpkins

bryanduxbury : http://svn.apache.org/viewvc/?view=rev&rev=1160931
Files : 
* /thrift/trunk/compiler/cpp/src/generate/t_cpp_generator.cc


>  thrift: wrap t_cpp_generator::generate_process_function() to 80
> ----------------------------------------------------------------
>
>                 Key: THRIFT-1283
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1283
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>            Reporter: Dave Watson
>            Assignee: Dave Watson
>            Priority: Minor
>             Fix For: 0.8
>
>         Attachments: 
> 0012-thrift-wrap-t_cpp_generator-generate_process_functio.patch
>
>
> From 91243efb4193412d54ccbd85ac991e035f80fa53 Mon Sep 17 00:00:00 2001
> From: Adam Simpkins <[email protected]>
> Date: Sat, 3 Apr 2010 02:49:43 +0000
> Subject: [PATCH 12/33] thrift: wrap
>  t_cpp_generator::generate_process_function() to 80
>  chars
> Summary:
> I'm planning on making some other changes to this function, and it will
> be easier to review them if the line-wrapping fixes are done separately.
> Test Plan:
> Built [three facebook projects] both with and without
> these changes, and made sure the resulting files were identical.
> Revert Plan:
> OK
> Conflicts:
>       compiler/cpp/src/generate/t_cpp_generator.cc
> ---
>  compiler/cpp/src/generate/t_cpp_generator.cc |  149 +++++++++++++++++--------
>  1 files changed, 101 insertions(+), 48 deletions(-)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to