From 3603e569db5a358cdb8ff7d645566e7718a815f9 Mon Sep 17 00:00:00 2001
From: 554325746@qq.com <554325746@qq.com>
Date: 星期三, 21 八月 2019 15:05:25 +0800
Subject: [PATCH] bug fix reverse

---
 shm.go |   48 +++++++++++++++++++++++++-----------------------
 1 files changed, 25 insertions(+), 23 deletions(-)

diff --git a/shm.go b/shm.go
index 16bc8f1..911c2d9 100644
--- a/shm.go
+++ b/shm.go
@@ -16,7 +16,7 @@
 
 // Send impl interface Diliver
 func (s *SHM) Send(data []byte) error {
-	if s.rw == nil {
+	if s == nil || s.rw == nil {
 		return errors.New("please init shm producer first")
 	}
 
@@ -31,7 +31,7 @@
 // Recv impl interface Diliver
 func (s *SHM) Recv() ([]byte, error) {
 
-	if s.rw == nil {
+	if s == nil || s.rw == nil {
 		return nil, errors.New("please open shm consumer first")
 	}
 
@@ -47,6 +47,9 @@
 
 // Close impl interface Deliver
 func (s *SHM) Close() {
+	if s == nil {
+		return
+	}
 	if s.rw != nil {
 		s.rw.Close()
 	}
@@ -55,54 +58,53 @@
 	}
 }
 
-func shmServer(m Mode, url string, args ...interface{}) *SHM {
+func shmServer(m Mode, url string, args ...interface{}) (*SHM, error) {
 	if m != Shm {
-		fmt.Println("this is not a shm mode: ", m)
-		return nil
+		return nil, errors.New("please use deliver.Shm mode")
 	}
 
 	var param []int
 	for _, v := range args {
-		fmt.Println(v)
 		switch v.(type) {
 		case int:
 			param = append(param, v.(int))
 		default:
-			fmt.Println("shmProducer recv error parameters")
 
-			return nil
+			return nil, errors.New("shmServer created recv error parameters")
 		}
 	}
-	if len(param) != 2 {
-		fmt.Println("shmProducer recv too much parameter: ", len(param))
-		return nil
+
+	blocks, size := 2, maxRecvSize
+	if len(param) == 2 {
+		blocks, size = param[0], param[1]
+		// return nil, errors.New("shmServer created recv too much parameters")
 	}
+
 	shm.Unlink(url)
-	if rw, err := shm.CreateSimplex(url, 0644, param[0], param[1]); err == nil {
+
+	rw, err := shm.CreateSimplex(url, 0644, blocks, size)
+	if err == nil {
 		fmt.Println(rw.Name())
 		return &SHM{
 			rw,
 			agent,
-		}
+		}, nil
 	}
 
-	fmt.Println("create simple shm error")
-	return nil
+	return nil, err
 }
 
-func shmClient(m Mode, url string, args ...interface{}) *SHM {
-
+func shmClient(m Mode, url string, args ...interface{}) (*SHM, error) {
 	if m != Shm {
-		fmt.Println("this is not a shm mode: ", m)
-		return nil
+		return nil, errors.New("please use deliver.Shm mode")
 	}
 
-	if rw, err := shm.OpenSimplex(url); err == nil {
+	rw, err := shm.OpenSimplex(url)
+	if err == nil {
 		return &SHM{
 			rw,
 			coactee,
-		}
+		}, nil
 	}
-	fmt.Println("shmConsumer open error")
-	return nil
+	return nil, err
 }

--
Gitblit v1.8.0