[thrift] branch master updated: THRIFT-4558 normalize name in C# constructor Client: csharp Patch: Pierre Mignolet

[jensg]

This is an automated email from the ASF dual-hosted git repository.

jensg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/thrift.git


The following commit(s) were added to refs/heads/master by this push:
     new 4a00fc6  THRIFT-4558 normalize name in C# constructor Client: csharp 
Patch: Pierre Mignolet <p.migno...@evs.com>
4a00fc6 is described below

commit 4a00fc658a1dc90be33a223a2c7ee336679e8745
Author: Pierre Mignolet <p.migno...@evs.com>
AuthorDate: Tue Apr 17 15:25:27 2018 +0200

    THRIFT-4558 normalize name in C# constructor
    Client: csharp
    Patch: Pierre Mignolet <p.migno...@evs.com>
    
    This closes #1546
---
 compiler/cpp/src/thrift/generate/t_csharp_generator.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/compiler/cpp/src/thrift/generate/t_csharp_generator.cc 
b/compiler/cpp/src/thrift/generate/t_csharp_generator.cc
index b108c45..cc2567b 100644
--- a/compiler/cpp/src/thrift/generate/t_csharp_generator.cc
+++ b/compiler/cpp/src/thrift/generate/t_csharp_generator.cc
@@ -843,7 +843,7 @@ void 
t_csharp_generator::generate_csharp_struct_definition(ofstream& out,
         } else {
           out << ", ";
         }
-        out << type_name((*m_iter)->get_type()) << " " << 
(*m_iter)->get_name();
+        out << type_name((*m_iter)->get_type()) << " " << 
normalize_name((*m_iter)->get_name());
       }
     }
     out << ") : this() {" << endl;
@@ -851,7 +851,7 @@ void 
t_csharp_generator::generate_csharp_struct_definition(ofstream& out,
 
     for (m_iter = members.begin(); m_iter != members.end(); ++m_iter) {
       if (field_is_required((*m_iter))) {
-        indent(out) << "this." << prop_name((*m_iter)) << " = " << 
(*m_iter)->get_name() << ";"
+        indent(out) << "this." << prop_name((*m_iter)) << " = " << 
normalize_name((*m_iter)->get_name()) << ";"
                     << endl;
       }
     }

-- 
To stop receiving notification emails like this one, please contact
je...@apache.org.