From: Rob Pike Date: Wed, 8 Apr 2015 18:20:15 +0000 (-0700) Subject: net/rpc: document that the type must be exported, not just the methods X-Git-Tag: go1.5beta1~1238 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=514eb4aa542f6989630b0cda290a77be3bc0fb4c;p=gostls13.git net/rpc: document that the type must be exported, not just the methods Fixes #10379. Change-Id: Ia4cdda36ed57a06371f9ace7365ce9e215228487 Reviewed-on: https://go-review.googlesource.com/8654 Reviewed-by: Rob Pike --- diff --git a/src/net/rpc/server.go b/src/net/rpc/server.go index 83728d55a1..6e6e881917 100644 --- a/src/net/rpc/server.go +++ b/src/net/rpc/server.go @@ -13,6 +13,7 @@ Only methods that satisfy these criteria will be made available for remote access; other methods will be ignored: + - the method's type is exported. - the method is exported. - the method has two arguments, both exported (or builtin) types. - the method's second argument is a pointer. @@ -216,7 +217,7 @@ func isExportedOrBuiltinType(t reflect.Type) bool { // Register publishes in the server the set of methods of the // receiver value that satisfy the following conditions: -// - exported method +// - exported method of exported type // - two arguments, both of exported type // - the second argument is a pointer // - one return value, of type error