From f4c71c73d0d3438d5eb8c72dc889fb292bc23cc8 Mon Sep 17 00:00:00 2001 From: Bryan Duxbury Date: Mon, 23 Feb 2009 17:49:24 +0000 Subject: [PATCH] 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 --- compiler/cpp/src/generate/t_java_generator.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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{ -- 2.17.1