review comments

Author: easwars

dialoptions.go

@@ -709,9 +709,7 @@ func WithIdleTimeout(d time.Duration) DialOption {
 // Deprecated: use experimental.WithRecvBufferPool instead.  Will be deleted in
 // v1.60.0 or later.
 func WithRecvBufferPool(bufferPool SharedBufferPool) DialOption {
-	return newFuncDialOption(func(o *dialOptions) {
-		o.recvBufferPool = bufferPool
-	})
+	return withRecvBufferPool(bufferPool)
 }
 
 func withRecvBufferPool(bufferPool SharedBufferPool) DialOption {

experimental/experimental.go

@@ -16,7 +16,7 @@
  *
  */
 
-// Package experimental is a placeholder for experimental features that might
+// Package experimental is a collection of experimental features that might
 // have some rough edges to them. Housing experimental features in this package
 // results in a user accessing these APIs as `experimental.Foo`, thereby making
 // it explicit that the feature is experimental and using them in production

server.go

@@ -584,9 +584,7 @@ func NumStreamWorkers(numServerWorkers uint32) ServerOption {
 // Deprecated: use experimental.WithRecvBufferPool instead.  Will be deleted in
 // v1.60.0 or later.
 func RecvBufferPool(bufferPool SharedBufferPool) ServerOption {
-	return newFuncServerOption(func(o *serverOptions) {
-		o.recvBufferPool = bufferPool
-	})
+	return recvBufferPool(bufferPool)
 }
 
 func recvBufferPool(bufferPool SharedBufferPool) ServerOption {