THRIFT-116. java: Isset fields for non-primitive types unnecessary
authorBryan Duxbury <bryanduxbury@apache.org>
Mon, 23 Feb 2009 17:49:24 +0000 (17:49 +0000)
committerBryan Duxbury <bryanduxbury@apache.org>
Mon, 23 Feb 2009 17:49:24 +0000 (17:49 +0000)
Prior commit on this issue was missing a !.

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@747090 13f79535-47bb-0310-9956-ffa450edef68

compiler/cpp/src/generate/t_java_generator.cc

index 0d2345a..a859894 100644 (file)
@@ -1014,7 +1014,7 @@ void t_java_generator::generate_java_validator(ofstream& out,
     if ((*f_iter)->get_req() == t_field::T_REQUIRED) {
       if (bean_style_) {
         out <<
-          indent() << "if (" << generate_isset_check(*f_iter) << ") {" << 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{