You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by jf...@apache.org on 2011/10/26 04:29:49 UTC

svn commit: r1189006 - /thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc

Author: jfarrell
Date: Wed Oct 26 02:29:48 2011
New Revision: 1189006

URL: http://svn.apache.org/viewvc?rev=1189006&view=rev
Log:
THRIFT-1406: Build error after applying THRIFT-1395
Client: delphi
Patch: Jens Geyer

Fixes build failure due to t_delphi_generator.cc:780: error: integer constant is too 
large for 'long' type


Modified:
    thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc

Modified: thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc
URL: http://svn.apache.org/viewvc/thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc?rev=1189006&r1=1189005&r2=1189006&view=diff
==============================================================================
--- thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc (original)
+++ thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc Wed Oct 26 02:29:48 2011
@@ -756,32 +756,16 @@ string t_delphi_generator::render_const_
         render << ((value->get_integer() > 0) ? "True" : "False");
         break;
       case t_base_type::TYPE_BYTE:
-        if( 0x7F < value->get_integer()) {
-          render << "ShortInt( " << value->get_integer() << ")";
-        } else {
-          render << value->get_integer();
-        }
+        render << "ShortInt( " << value->get_integer() << ")";
         break;
       case t_base_type::TYPE_I16:
-        if( 0x7FFF < value->get_integer()) {
-          render << "SmallInt( " << value->get_integer() << ")";
-        } else {
-          render << value->get_integer();
-        }
+        render << "SmallInt( " << value->get_integer() << ")";
         break;
       case t_base_type::TYPE_I32:
-        if( 0x7FFFFFFF < value->get_integer()) {
-          render << "LongInt( " << value->get_integer() << ")";
-        } else {
-          render << value->get_integer();
-        }
+        render << "LongInt( " << value->get_integer() << ")";
         break;
       case t_base_type::TYPE_I64:
-        if( 0x7FFFFFFFFFFFFFFF < value->get_integer()) {
-          render << "Int64( " << value->get_integer() << ")";
-        } else {
-          render << value->get_integer();
-        }
+        render << "Int64( " << value->get_integer() << ")";
         break;
       case t_base_type::TYPE_DOUBLE:
         if (value->get_type() == t_const_value::CV_INTEGER) {