From: Bryan Duxbury <bryanduxbury@apache.org>
Date: Mon, 23 Feb 2009 17:49:24 +0000 (+0000)
Subject: THRIFT-116. java: Isset fields for non-primitive types unnecessary
X-Git-Tag: 0.2.0~286
X-Git-Url: https://source.supwisdom.com/gerrit/gitweb?a=commitdiff_plain;h=f4c71c73d0d3438d5eb8c72dc889fb292bc23cc8;p=common%2Fthrift.git

THRIFT-116. java: Isset fields for non-primitive types unnecessary

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
---

diff --git a/compiler/cpp/src/generate/t_java_generator.cc b/compiler/cpp/src/generate/t_java_generator.cc
index 0d2345a9..a859894e 100644
--- a/compiler/cpp/src/generate/t_java_generator.cc
+++ b/compiler/cpp/src/generate/t_java_generator.cc
@@ -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{