Code review changes

clientconn.go

@@ -1980,13 +1980,14 @@ func (cc *ClientConn) determineAuthority() error {
 		return fmt.Errorf("ClientConn's authority from transport creds %q and dial option %q don't match", authorityFromCreds, authorityFromDialOption)
 	}
 
+	endpoint := cc.parsedTarget.Endpoint()
 	if authorityFromDialOption != "" {
 		cc.authority = authorityFromDialOption
 	} else if authorityFromCreds != "" {
 		cc.authority = authorityFromCreds
 	} else if auth, ok := cc.resolverBuilder.(resolver.AuthorityOverrider); ok {
 		cc.authority = auth.OverrideAuthority(cc.parsedTarget)
-	} else if endpoint := cc.parsedTarget.Endpoint(); strings.HasPrefix(endpoint, ":") {
+	} else if strings.HasPrefix(endpoint, ":") {
 		cc.authority = "localhost" + endpoint
 	} else {
 		cc.authority = encodeAuthority(endpoint)

resolver/resolver.go

@@ -322,7 +322,7 @@ type Resolver interface {
 
 // AuthorityOverrider is implemented by Builders that wish to override the
 // default authority for the ClientConn.
-// By default, the authority used is target.Endpoint()
+// By default, the authority used is target.Endpoint().
 type AuthorityOverrider interface {
 	// OverrideAuthority returns the authority to use for a ClientConn with the
 	// given target. It must not perform I/O or any other blocking operations.