THRIFT-4914: Fix name redeclaration bug in compiled go code

Client: go

This fixes the bug reported in
https://github.com/apache/thrift/pull/2315#discussion_r577919697.

1 files changed, 14 insertions, 0 deletions

diff --git a/test/ThriftTest.thrift b/test/ThriftTest.thrift
index ac49aee01..c88187c54 100644
--- a/test/ThriftTest.thrift
+++ b/test/ThriftTest.thrift
@@ -411,3 +411,17 @@ struct StructB {
 struct OptionalSetDefaultTest {
   1: optional set<string> with_default = [ "test" ]
 }
+
+service ConflictingNamesTest {
+  /**
+   * Use some names that could conflict with the compiler code as args
+   * to make sure that the compiler handled them correctly.
+   */
+  void testNameConflicts(
+    // 1: string args,   // args is already a reserved keyword in thrift compiler
+    // 2: string result, // result will cause problems in compiled netstd code
+    3: string meta,
+    4: string r,
+    5: string err,
+  )
+}