From: Roger Meier <roger@apache.org>
Date: Fri, 13 Apr 2012 14:33:56 +0000 (+0000)
Subject: THRIFT-1565 Thrift/cpp: Use default constructor for strings
X-Git-Tag: 0.9.1~403
X-Git-Url: https://source.supwisdom.com/gerrit/gitweb?a=commitdiff_plain;h=629d5a9bcf49ee6e38049b7849cd4db24ef86632;p=common%2Fthrift.git

THRIFT-1565 Thrift/cpp: Use default constructor for strings
Patch: Dave Watson

git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1325785 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/compiler/cpp/src/generate/t_cpp_generator.cc b/compiler/cpp/src/generate/t_cpp_generator.cc
index 3bc6a81c..08c08844 100755
--- a/compiler/cpp/src/generate/t_cpp_generator.cc
+++ b/compiler/cpp/src/generate/t_cpp_generator.cc
@@ -865,7 +865,7 @@ void t_cpp_generator::generate_struct_definition(ofstream& out,
         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_field* tfield, bool init, bool pointer,
       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";