commit 2b1b32c2836c21ce012d39ebc02c8488d67d4cef
author D. Can Celasun <can@dcc.im> Tue May 30 12:44:56 2017 +0200
committer Jens Geyer <jensg@apache.org> Wed May 31 22:56:52 2017 +0200
tree bdd8b84abd1eae8cdf7e458b5f304075606de84b
parent 197b0629935b9b240915485dd3ed631b19b74d7d

THRIFT-3703 Unions Field Count Does Not Consider Map/Set/List Fields
Client: Go
Patch: D. Can Celasun <can@dcc.im>

This closes #1281

compiler/cpp/src/thrift/generate/t_go_generator.cc

diff --git a/compiler/cpp/src/thrift/generate/t_go_generator.cc b/compiler/cpp/src/thrift/generate/t_go_generator.cc
index c7f5279..8c8bda7 100644
--- a/compiler/cpp/src/thrift/generate/t_go_generator.cc
+++ b/compiler/cpp/src/thrift/generate/t_go_generator.cc
@@ -1445,7 +1445,9 @@
     if ((*f_iter)->get_req() == t_field::T_REQUIRED)
       continue;
 
-    if (!is_pointer_field(*f_iter))
+    t_type* type = (*f_iter)->get_type()->get_true_type();
+
+    if (!(is_pointer_field(*f_iter) || type->is_map() || type->is_set() || type->is_list()))
       continue;
 
     const string field_name(publicize(escape_string((*f_iter)->get_name())));