xdsclient: change receiver on BootstrapConfig() to be consistent (#5532)

easwars · web-flow · commit 9a689dc4bb1a · 2022-07-25T14:16:05.000-07:00
diff --git a/xds/internal/xdsclient/client.go b/xds/internal/xdsclient/client.go
@@ -52,6 +52,8 @@ type XDSClient interface {
 	Close()
 }
 
+var _ XDSClient = &clientImpl{}
+
 // clientImpl is the real implementation of the xds client. The exported Client
 // is a wrapper of this struct with a ref count.
 //
@@ -105,7 +107,7 @@ func newWithConfig(config *bootstrap.Config, watchExpiryTimeout time.Duration, i
 
 // BootstrapConfig returns the configuration read from the bootstrap file.
 // Callers must treat the return value as read-only.
-func (c *clientRefCounted) BootstrapConfig() *bootstrap.Config {
+func (c *clientImpl) BootstrapConfig() *bootstrap.Config {
 	return c.config
 }
 

Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,8 @@ type XDSClient interface {`
`52`	`52`	`Close()`
`53`	`53`	`}`
`54`	`54`
	`55`	`+var _ XDSClient = &clientImpl{}`
	`56`	`+`
`55`	`57`	`// clientImpl is the real implementation of the xds client. The exported Client`
`56`	`58`	`// is a wrapper of this struct with a ref count.`
`57`	`59`	`//`
`@@ -105,7 +107,7 @@ func newWithConfig(config *bootstrap.Config, watchExpiryTimeout time.Duration, i`
`105`	`107`
`106`	`108`	`// BootstrapConfig returns the configuration read from the bootstrap file.`
`107`	`109`	`// Callers must treat the return value as read-only.`
`108`		`-func (c clientRefCounted) BootstrapConfig() bootstrap.Config {`
	`110`	`+func (c clientImpl) BootstrapConfig() bootstrap.Config {`
`109`	`111`	`return c.config`
`110`	`112`	`}`
`111`	`113`