THRIFT-572. java: Union compareTo is broken
authorBryan Duxbury <bryanduxbury@apache.org>
Mon, 21 Sep 2009 16:52:48 +0000 (16:52 +0000)
committerBryan Duxbury <bryanduxbury@apache.org>
Mon, 21 Sep 2009 16:52:48 +0000 (16:52 +0000)
git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@817317 13f79535-47bb-0310-9956-ffa450edef68

compiler/cpp/src/generate/t_java_generator.cc

index 427e654..4166f3c 100644 (file)
@@ -960,7 +960,7 @@ void t_java_generator::generate_union_comparisons(ofstream& out, t_struct* tstru
     indent(out) << "@Override" << endl;
     indent(out) << "public int compareTo(" << type_name(tstruct) << " other) {" << endl;
     indent(out) << "  int lastComparison = TBaseHelper.compareTo(getSetField(), other.getSetField());" << endl;
-    indent(out) << "  if (lastComparison != 0) {" << endl;
+    indent(out) << "  if (lastComparison == 0) {" << endl;
     indent(out) << "    return TBaseHelper.compareTo((Comparable)getFieldValue(), (Comparable)other.getFieldValue());" << endl;
     indent(out) << "  }" << endl;
     indent(out) << "  return lastComparison;" << endl;