You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by jf...@apache.org on 2012/05/16 05:49:57 UTC
svn commit: r1338995 -
/thrift/trunk/compiler/cpp/src/generate/t_cocoa_generator.cc
Author: jfarrell
Date: Wed May 16 03:49:57 2012
New Revision: 1338995
URL: http://svn.apache.org/viewvc?rev=1338995&view=rev
Log:
Thrift-541: Cocoa code generator doesn't put keywords before all arguments.
Client: cocoa
Patch: Andrew McGeachie
Updates args list to contain keywords before any args.
Modified:
thrift/trunk/compiler/cpp/src/generate/t_cocoa_generator.cc
Modified: thrift/trunk/compiler/cpp/src/generate/t_cocoa_generator.cc
URL: http://svn.apache.org/viewvc/thrift/trunk/compiler/cpp/src/generate/t_cocoa_generator.cc?rev=1338995&r1=1338994&r2=1338995&view=diff
==============================================================================
--- thrift/trunk/compiler/cpp/src/generate/t_cocoa_generator.cc (original)
+++ thrift/trunk/compiler/cpp/src/generate/t_cocoa_generator.cc Wed May 16 03:49:57 2012
@@ -2544,12 +2544,14 @@ string t_cocoa_generator::argument_list(
vector<t_field*>::const_iterator f_iter;
bool first = true;
for (f_iter = fields.begin(); f_iter != fields.end(); ++f_iter) {
+ string argPrefix = "";
if (first) {
first = false;
} else {
+ argPrefix = (*f_iter)->get_name();
result += " ";
}
- result += ": (" + type_name((*f_iter)->get_type()) + ") " + (*f_iter)->get_name();
+ result += argPrefix + ": (" + type_name((*f_iter)->get_type()) + ") " + (*f_iter)->get_name();
}
return result;
}