From 4338cf10114f5176d120719cbc51bdc2177eae1c Mon Sep 17 00:00:00 2001
From: zhangmeng <775834166@qq.com>
Date: 星期一, 26 八月 2019 16:05:59 +0800
Subject: [PATCH] bug fix
---
shm.go | 66 +++++++++++++++++----------------
1 files changed, 34 insertions(+), 32 deletions(-)
diff --git a/shm.go b/shm.go
index 21f8167..727d18c 100644
--- a/shm.go
+++ b/shm.go
@@ -4,20 +4,20 @@
"errors"
"fmt"
"io"
+ "time"
- "github.com/tmthrgd/shm-go"
+ "basic.com/valib/shm.git"
)
// SHM share memory
type SHM struct {
- rw *shm.ReadWriteCloser
- typ td
- data []byte
+ rw *shm.ReadWriteCloser
+ typ td
}
// 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")
}
@@ -32,14 +32,15 @@
// 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")
}
- n, err := s.rw.Read(s.data)
+ data := make([]byte, maxRecvSize)
+ n, err := s.rw.Read(data)
if err == nil || err == io.EOF {
- s.data = s.data[:n:n]
- return s.data, nil
+ data = data[:n:n]
+ return data, nil
}
return nil, err
@@ -47,6 +48,9 @@
// Close impl interface Deliver
func (s *SHM) Close() {
+ if s == nil {
+ return
+ }
if s.rw != nil {
s.rw.Close()
}
@@ -55,56 +59,54 @@
}
}
-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")
}
+
+ time.Sleep(time.Millisecond)
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,
- }
+ }, 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,
- make([]byte, maxRecvSize),
- }
+ }, nil
}
- fmt.Println("shmConsumer open error")
- return nil
+ return nil, err
}
--
Gitblit v1.8.0