Merge pull request #4 from letsfire/develop

update return value
letsfire · Jan 26, 2019 · a8d605e · a8d605e
2 parents 9a20a2f + 1051f7f
commit a8d605e
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 6 deletions.
diff --git a/mode/alone/alone.go b/mode/alone/alone.go
@@ -17,7 +17,9 @@ func (sam *standAloneMode) NewConn() (redis.Conn, error) {
 	return sam.pool.Dial()
 }
 
-func New(optFuncs ...OptFunc) mode.IMode {
+var _ mode.IMode = &standAloneMode{}
+
+func New(optFuncs ...OptFunc) *standAloneMode {
 	opts := options{
 		addr:     "127.0.0.1:6379",
 		dialOpts: mode.DefaultDialOpts(),

diff --git a/mode/cluster/cluster.go b/mode/cluster/cluster.go
@@ -18,7 +18,9 @@ func (cm *clusterMode) NewConn() (redis.Conn, error) {
 	return cm.rc.Dial()
 }
 
-func New(optFuncs ...OptFunc) mode.IMode {
+var _ mode.IMode = &clusterMode{}
+
+func New(optFuncs ...OptFunc) *clusterMode {
 	opts := options{
 		nodes: []string{
 			"127.0.0.1:30001", "127.0.0.1:30002", "127.0.0.1:30003",

diff --git a/mode/mode.go b/mode/mode.go
@@ -3,7 +3,7 @@ package mode
 import (
 	"runtime"
 	"time"
-	
+
 	"github.com/gomodule/redigo/redis"
 )
 

diff --git a/mode/sentinel/sentinel.go b/mode/sentinel/sentinel.go
@@ -3,7 +3,7 @@ package sentinel
 import (
 	"runtime"
 	"time"
-	
+
 	"github.com/FZambia/sentinel"
 	"github.com/gomodule/redigo/redis"
 	"github.com/letsfire/redigo/mode"
@@ -21,7 +21,9 @@ func (sm *sentinelMode) NewConn() (redis.Conn, error) {
 	return sm.pool.Dial()
 }
 
-func New(optFuncs ...OptFunc) mode.IMode {
+var _ mode.IMode = &sentinelMode{}
+
+func New(optFuncs ...OptFunc) *sentinelMode {
 	opts := options{
 		addrs:      []string{"127.0.0.1:26379"},
 		masterName: "mymaster",

diff --git a/redigo_test.go b/redigo_test.go
@@ -4,7 +4,7 @@ import (
 	"sync/atomic"
 	"testing"
 	"time"
-	
+
 	"github.com/gomodule/redigo/redis"
 	"github.com/letsfire/redigo/mode/alone"
 	"github.com/letsfire/redigo/mode/cluster"