THRIFT-1565 Thrift/cpp: Use default constructor for strings
authorRoger Meier <roger@apache.org>
Fri, 13 Apr 2012 14:33:56 +0000 (14:33 +0000)
committerRoger Meier <roger@apache.org>
Fri, 13 Apr 2012 14:33:56 +0000 (14:33 +0000)
Patch: Dave Watson

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

compiler/cpp/src/generate/t_cpp_generator.cc

index 3bc6a81..08c0884 100755 (executable)
@@ -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";