Add reflect package to collisions map

Summary:
We will begin using "reflect" downstream.
Ensure that collisions are handled properly.

Reviewed By: leoleovich

Differential Revision: D63477977

fbshipit-source-id: 1ebf50ebafcd636faaa4762a98c7bd8eb74fab06

thrift/compiler/generate/templates/go/svcs.go.mustache

@@ -21,6 +21,7 @@
 import (
     "context"
     "fmt"
+    "reflect"
     "strings"
 
     {{#program:thrift_imports}}
@@ -38,6 +39,7 @@ import (
 // (needed to ensure safety because of naive import list construction)
 var _ = context.Background
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 {{#program:gen_metadata?}}

thrift/compiler/generate/templates/go/types.go.mustache

@@ -18,6 +18,7 @@
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     {{#program:thrift_imports}}
@@ -29,6 +30,7 @@ import (
 {{> common/unused_imports_protection}}
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/lib/go/util.h

@@ -99,6 +99,7 @@ class codegen_data {
       {"sync", 0},
       {"metadata", 0},
       {"maps", 0},
+      {"reflect", 0},
   };
 
   struct go_munged_names_cache_key_ {

thrift/compiler/test/fixtures/adapter/out/go/gen-go/module/svcs.go

@@ -9,6 +9,7 @@ package module
 import (
     "context"
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
@@ -18,6 +19,7 @@ import (
 // (needed to ensure safety because of naive import list construction)
 var _ = context.Background
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 var _ = metadata.GoUnusedProtection__

thrift/compiler/test/fixtures/adapter/out/go/gen-go/module/types.go

@@ -7,13 +7,15 @@ package module
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/adapter/out/go/gen-go/thrift/annotation/cpp/types.go

@@ -7,13 +7,15 @@ package cpp
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/adapter/out/go/gen-go/thrift/annotation/hack/types.go

@@ -7,13 +7,15 @@ package hack
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/adapter/out/go/gen-go/thrift/annotation/python/types.go

@@ -7,13 +7,15 @@ package python
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/adapter/out/go/gen-go/thrift/annotation/rust/types.go

@@ -7,13 +7,15 @@ package rust
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/adapter/out/go/gen-go/thrift/annotation/scope/types.go

@@ -7,13 +7,15 @@ package scope
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/adapter/out/go/gen-go/thrift/annotation/thrift/types.go

@@ -7,13 +7,15 @@ package thrift
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic-annotations/out/go/gen-go/module/svcs.go

@@ -9,6 +9,7 @@ package module
 import (
     "context"
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
@@ -18,6 +19,7 @@ import (
 // (needed to ensure safety because of naive import list construction)
 var _ = context.Background
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 var _ = metadata.GoUnusedProtection__

thrift/compiler/test/fixtures/basic-annotations/out/go/gen-go/module/types.go

@@ -7,13 +7,15 @@ package module
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic-annotations/out/go/gen-go/thrift/annotation/cpp/types.go

@@ -7,13 +7,15 @@ package cpp
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic-annotations/out/go/gen-go/thrift/annotation/go/types.go

@@ -7,13 +7,15 @@ package go_
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic-annotations/out/go/gen-go/thrift/annotation/hack/types.go

@@ -7,13 +7,15 @@ package hack
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic-annotations/out/go/gen-go/thrift/annotation/scope/types.go

@@ -7,13 +7,15 @@ package scope
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic-annotations/out/go/gen-go/thrift/annotation/thrift/types.go

@@ -7,13 +7,15 @@ package thrift
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic/out/go/gen-go/module/svcs.go

@@ -9,6 +9,7 @@ package module
 import (
     "context"
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
@@ -18,6 +19,7 @@ import (
 // (needed to ensure safety because of naive import list construction)
 var _ = context.Background
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 var _ = metadata.GoUnusedProtection__

thrift/compiler/test/fixtures/basic/out/go/gen-go/module/types.go

@@ -7,13 +7,15 @@ package module
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic/out/go/gen-go/thrift/annotation/hack/types.go

@@ -7,13 +7,15 @@ package hack
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic/out/go/gen-go/thrift/annotation/scope/types.go

@@ -7,13 +7,15 @@ package scope
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/basic/out/go/gen-go/thrift/annotation/thrift/types.go

@@ -7,13 +7,15 @@ package thrift
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/complex-union/out/go/gen-go/module/types.go

@@ -7,13 +7,15 @@ package module
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO
 

thrift/compiler/test/fixtures/complex-union/out/go/gen-go/thrift/annotation/cpp/types.go

@@ -7,13 +7,15 @@ package cpp
 
 import (
     "fmt"
+    "reflect"
     "strings"
 
     thrift "github.com/facebook/fbthrift/thrift/lib/go/thrift/types"
 )
 
 // (needed to ensure safety because of naive import list construction)
 var _ = fmt.Printf
+var _ = reflect.Ptr
 var _ = strings.Split
 var _ = thrift.ZERO