Prefix php names with service_ to avoid naming conflicts and use False not false in python
git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@664811 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/compiler/cpp/src/generate/t_php_generator.cc b/compiler/cpp/src/generate/t_php_generator.cc
index de816dc..3507a70 100644
--- a/compiler/cpp/src/generate/t_php_generator.cc
+++ b/compiler/cpp/src/generate/t_php_generator.cc
@@ -368,8 +368,11 @@
for (f_iter = functions.begin(); f_iter != functions.end(); ++f_iter) {
t_struct* ts = (*f_iter)->get_arglist();
+ string name = ts->get_name();
+ ts->set_name(service_name_ + "_" + name);
generate_php_struct_definition(f_service_, ts, false);
generate_php_function_helpers(*f_iter);
+ ts->set_name(name);
}
}
@@ -379,7 +382,7 @@
* @param tfunction The function
*/
void t_php_generator::generate_php_function_helpers(t_function* tfunction) {
- t_struct result(tfunction->get_name() + "_result");
+ t_struct result(service_name_ + "_" + tfunction->get_name() + "_result");
t_field success(tfunction->get_returntype(), "success", 0);
if (!tfunction->get_returntype()->is_void()) {
result.append(&success);
@@ -514,7 +517,7 @@
"public function send_" << function_signature(*f_iter) << endl;
scope_up(f_service_);
- std::string argsname = (*f_iter)->get_name() + "_args";
+ std::string argsname = service_name_ + "_" + (*f_iter)->get_name() + "_args";
// Serialize the request header
if (binary_inline_) {
@@ -556,7 +559,7 @@
if (!(*f_iter)->is_async()) {
- std::string resultname = (*f_iter)->get_name() + "_result";
+ std::string resultname = service_name_ + "_" + (*f_iter)->get_name() + "_result";
t_struct noargs;
t_function recv_function((*f_iter)->get_returntype(),
diff --git a/compiler/cpp/src/generate/t_py_generator.cc b/compiler/cpp/src/generate/t_py_generator.cc
index f07af33..86c2bd0 100644
--- a/compiler/cpp/src/generate/t_py_generator.cc
+++ b/compiler/cpp/src/generate/t_py_generator.cc
@@ -1212,7 +1212,7 @@
result += " = ''";
break;
case t_base_type::TYPE_BOOL:
- result += " = false";
+ result += " = False";
break;
case t_base_type::TYPE_BYTE:
case t_base_type::TYPE_I16: