fixup! *: plug in no-op metric implementations for nil fields

Author: suj-krishnan

drpcconn/conn.go

@@ -30,7 +30,7 @@ type Options struct {
 	// rpcs it creates.
 	CollectStats bool
 
-	// CollectMetrics controls whether the client should collect metrics
+	// CollectMetrics controls whether the client should collect metrics.
 	CollectMetrics bool
 
 	// Metrics holds optional metrics the client will populate.

drpcpool/pool.go

@@ -36,8 +36,7 @@ type Options struct {
 	// no values for any single key.
 	KeyCapacity int
 
-	// Metrics holds optional metrics the pool will populate. If nil,
-	// no metrics are recorded.
+	// Metrics holds optional metrics the pool will populate.
 	Metrics PoolMetrics
 
 	// Labels holds optional labels to be attached to all metrics.

drpcserver/server.go

@@ -46,8 +46,7 @@ type Options struct {
 	// restrictions. If it returns a non-nil error the connection is rejected.
 	TLSCipherRestrict func(conn net.Conn) error
 
-	// Metrics holds optional metrics the server will populate. If nil, no
-	// metrics are recorded.
+	// Metrics holds optional metrics the server will populate.
 	Metrics ServerMetrics
 }