]> Cypherpunks repositories - gostls13.git/commitdiff
net/rpc: document that the type must be exported, not just the methods
authorRob Pike <r@golang.org>
Wed, 8 Apr 2015 18:20:15 +0000 (11:20 -0700)
committerRob Pike <r@golang.org>
Wed, 8 Apr 2015 18:34:35 +0000 (18:34 +0000)
Fixes #10379.

Change-Id: Ia4cdda36ed57a06371f9ace7365ce9e215228487
Reviewed-on: https://go-review.googlesource.com/8654
Reviewed-by: Rob Pike <r@golang.org>
src/net/rpc/server.go

index 83728d55a18b4e509a2e9e575ddce41381d2ac16..6e6e881917494effee20b2deae4d1057523bac97 100644 (file)
@@ -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