You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by ro...@apache.org on 2012/04/13 16:38:20 UTC
svn commit: r1325786 -
/thrift/trunk/compiler/cpp/src/generate/t_py_generator.cc
Author: roger
Date: Fri Apr 13 14:38:19 2012
New Revision: 1325786
URL: http://svn.apache.org/viewvc?rev=1325786&view=rev
Log:
THRIFT-1570 Correctly generate code for empty services.
Patch: Dave Watson
Modified:
thrift/trunk/compiler/cpp/src/generate/t_py_generator.cc
Modified: thrift/trunk/compiler/cpp/src/generate/t_py_generator.cc
URL: http://svn.apache.org/viewvc/thrift/trunk/compiler/cpp/src/generate/t_py_generator.cc?rev=1325786&r1=1325785&r2=1325786&view=diff
==============================================================================
--- thrift/trunk/compiler/cpp/src/generate/t_py_generator.cc (original)
+++ thrift/trunk/compiler/cpp/src/generate/t_py_generator.cc Fri Apr 13 14:38:19 2012
@@ -1539,10 +1539,13 @@ void t_py_generator::generate_service_re
f_remote << endl;
}
- f_remote << "else:" << endl;
- f_remote << " print 'Unrecognized method %s' % cmd" << endl;
- f_remote << " sys.exit(1)" << endl;
- f_remote << endl;
+
+ if (functions.size() > 0) {
+ f_remote << "else:" << endl;
+ f_remote << " print 'Unrecognized method %s' % cmd" << endl;
+ f_remote << " sys.exit(1)" << endl;
+ f_remote << endl;
+ }
f_remote << "transport.close()" << endl;