Author: roger
Date: Fri Apr 13 14:33:56 2012
New Revision: 1325785

URL: http://svn.apache.org/viewvc?rev=1325785&view=rev
Log:
THRIFT-1565 Thrift/cpp: Use default constructor for strings
Patch: Dave Watson

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

Modified: thrift/trunk/compiler/cpp/src/generate/t_cpp_generator.cc
URL: 
http://svn.apache.org/viewvc/thrift/trunk/compiler/cpp/src/generate/t_cpp_generator.cc?rev=1325785&r1=1325784&r2=1325785&view=diff
==============================================================================
--- thrift/trunk/compiler/cpp/src/generate/t_cpp_generator.cc (original)
+++ thrift/trunk/compiler/cpp/src/generate/t_cpp_generator.cc Fri Apr 13 
14:33:56 2012
@@ -865,7 +865,7 @@ void t_cpp_generator::generate_struct_de
         if (t->is_enum()) {
           dval += "(" + type_name(t) + ")";
         }
-        dval += t->is_string() ? "\"\"" : "0";
+        dval += t->is_string() ? "" : "0";
         t_const_value* cv = (*m_iter)->get_value();
         if (cv != NULL) {
           dval = render_const_value(out, (*m_iter)->get_name(), t, cv);
@@ -4372,9 +4372,7 @@ string t_cpp_generator::declare_field(t_
       t_base_type::t_base tbase = ((t_base_type*)type)->get_base();
       switch (tbase) {
       case t_base_type::TYPE_VOID:
-        break;
       case t_base_type::TYPE_STRING:
-        result += " = \"\"";
         break;
       case t_base_type::TYPE_BOOL:
         result += " = false";


Reply via email to