commit a8a97bde9eeada5ce0571ea1650b18f3ebc50e42
author Roman Soroka <romanoid@uber.com> Thu Mar 22 00:07:23 2018 -0700
committer James E. King III <jking@apache.org> Thu Mar 22 11:30:14 2018 -0400
tree 0aa7c105ea46b7f518b6673f7ffff2fe50df4b4b
parent 714c77cebe72659f6577eb9c5293693bdfc9e692

THRIFT-4513: fix bug in comparator introduced by e58f75d

This closes #1517

compiler/cpp/src/thrift/parse/t_const_value.h

diff --git a/compiler/cpp/src/thrift/parse/t_const_value.h b/compiler/cpp/src/thrift/parse/t_const_value.h
index f4416e9..6a114cf 100644
--- a/compiler/cpp/src/thrift/parse/t_const_value.h
+++ b/compiler/cpp/src/thrift/parse/t_const_value.h
@@ -159,10 +159,10 @@
       if (*(left.first) < *(right.first)) {
         return true;
       } else {
-        if (right.first < left.first) {
-          return *(left.second) < *(right.second);
-        } else {
+        if (*(right.first) < *(left.first)) {
           return false;
+        } else {
+          return *(left.second) < *(right.second);
         }
       }
     }