Author: bryanduxbury
Date: Sat Feb 21 01:57:46 2009
New Revision: 746429
URL: http://svn.apache.org/viewvc?rev=746429&view=rev
Log:
THRIFT-116. java: Isset fields for non-primitive types unnecessary
Missed one of the cases where isset was being used on non-primitive fields.
Modified:
incubator/thrift/trunk/compiler/cpp/src/generate/t_java_generator.cc
Modified: incubator/thrift/trunk/compiler/cpp/src/generate/t_java_generator.cc
URL:
http://svn.apache.org/viewvc/incubator/thrift/trunk/compiler/cpp/src/generate/t_java_generator.cc?rev=746429&r1=746428&r2=746429&view=diff
==============================================================================
--- incubator/thrift/trunk/compiler/cpp/src/generate/t_java_generator.cc
(original)
+++ incubator/thrift/trunk/compiler/cpp/src/generate/t_java_generator.cc Sat
Feb 21 01:57:46 2009
@@ -1014,7 +1014,7 @@
if ((*f_iter)->get_req() == t_field::T_REQUIRED) {
if (bean_style_) {
out <<
- indent() << "if (!__isset." << (*f_iter)->get_name() << ") {" <<
endl <<
+ indent() << "if (" << generate_isset_check(*f_iter) << ") {" << endl
<<
indent() << " throw new TProtocolException(\"Required field '" <<
(*f_iter)->get_name() << "' is unset! Struct:\" + toString());" << endl <<
indent() << "}" << endl << endl;
} else{
