Skip to content

Commit 7c9a61a

Browse files
committed
opt: rename eventloop.closeConn to eventloop.close
1 parent 9d6eed1 commit 7c9a61a

File tree

4 files changed

+15
-15
lines changed

4 files changed

+15
-15
lines changed

connection_bsd.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ import (
2626
func (c *conn) handleEvents(_ int, filter int16) (err error) {
2727
switch filter {
2828
case netpoll.EVFilterSock:
29-
err = c.loop.closeConn(c, unix.ECONNRESET)
29+
err = c.loop.close(c, unix.ECONNRESET)
3030
case netpoll.EVFilterWrite:
3131
if !c.outboundBuffer.IsEmpty() {
3232
err = c.loop.write(c)

connection_unix.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ func (c *conn) write(data []byte) (n int, err error) {
134134
err = c.loop.poller.ModReadWrite(&c.pollAttachment)
135135
return
136136
}
137-
if err := c.loop.closeConn(c, os.NewSyscallError("write", err)); err != nil {
137+
if err := c.loop.close(c, os.NewSyscallError("write", err)); err != nil {
138138
logging.Errorf("failed to close connection(fd=%d,peer=%+v) on conn.write: %v",
139139
c.fd, c.remoteAddr, err)
140140
}
@@ -168,7 +168,7 @@ func (c *conn) writev(bs [][]byte) (n int, err error) {
168168
err = c.loop.poller.ModReadWrite(&c.pollAttachment)
169169
return
170170
}
171-
if err := c.loop.closeConn(c, os.NewSyscallError("writev", err)); err != nil {
171+
if err := c.loop.close(c, os.NewSyscallError("writev", err)); err != nil {
172172
logging.Errorf("failed to close connection(fd=%d,peer=%+v) on conn.writev: %v",
173173
c.fd, c.remoteAddr, err)
174174
}
@@ -467,7 +467,7 @@ func (c *conn) Wake(callback AsyncCallback) error {
467467

468468
func (c *conn) CloseWithCallback(callback AsyncCallback) error {
469469
return c.loop.poller.Trigger(func(_ interface{}) (err error) {
470-
err = c.loop.closeConn(c, nil)
470+
err = c.loop.close(c, nil)
471471
if callback != nil {
472472
_ = callback(c, err)
473473
}
@@ -477,7 +477,7 @@ func (c *conn) CloseWithCallback(callback AsyncCallback) error {
477477

478478
func (c *conn) Close() error {
479479
return c.loop.poller.Trigger(func(_ interface{}) (err error) {
480-
err = c.loop.closeConn(c, nil)
480+
err = c.loop.close(c, nil)
481481
return
482482
}, nil)
483483
}

eventloop_unix.go

+8-8
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ func (el *eventloop) countConn() int32 {
5656
func (el *eventloop) closeConns() {
5757
// Close loops and all outstanding connections
5858
el.connections.iterate(func(c *conn) bool {
59-
_ = el.closeConn(c, nil)
59+
_ = el.close(c, nil)
6060
return true
6161
})
6262
}
@@ -105,15 +105,15 @@ func (el *eventloop) read(c *conn) error {
105105
if n == 0 {
106106
err = unix.ECONNRESET
107107
}
108-
return el.closeConn(c, os.NewSyscallError("read", err))
108+
return el.close(c, os.NewSyscallError("read", err))
109109
}
110110

111111
c.buffer = el.buffer[:n]
112112
action := el.eventHandler.OnTraffic(c)
113113
switch action {
114114
case None:
115115
case Close:
116-
return el.closeConn(c, nil)
116+
return el.close(c, nil)
117117
case Shutdown:
118118
return gerrors.ErrEngineShutdown
119119
}
@@ -144,7 +144,7 @@ func (el *eventloop) write(c *conn) error {
144144
case unix.EAGAIN:
145145
return nil
146146
default:
147-
return el.closeConn(c, os.NewSyscallError("write", err))
147+
return el.close(c, os.NewSyscallError("write", err))
148148
}
149149

150150
// All data have been drained, it's no need to monitor the writable events,
@@ -156,7 +156,7 @@ func (el *eventloop) write(c *conn) error {
156156
return nil
157157
}
158158

159-
func (el *eventloop) closeConn(c *conn, err error) (rerr error) {
159+
func (el *eventloop) close(c *conn, err error) (rerr error) {
160160
if addr := c.localAddr; addr != nil && strings.HasPrefix(c.localAddr.Network(), "udp") {
161161
rerr = el.poller.Delete(c.fd)
162162
if c.fd != el.ln.fd {
@@ -182,7 +182,7 @@ func (el *eventloop) closeConn(c *conn, err error) (rerr error) {
182182
iov = iov[:iovMax]
183183
}
184184
if n, e := io.Writev(c.fd, iov); e != nil {
185-
el.getLogger().Warnf("closeConn: error occurs when sending data back to peer, %v", e)
185+
el.getLogger().Warnf("close: error occurs when sending data back to peer, %v", e)
186186
break
187187
} else { //nolint:revive
188188
_, _ = c.outboundBuffer.Discard(n)
@@ -263,7 +263,7 @@ func (el *eventloop) handleAction(c *conn, action Action) error {
263263
case None:
264264
return nil
265265
case Close:
266-
return el.closeConn(c, nil)
266+
return el.close(c, nil)
267267
case Shutdown:
268268
return gerrors.ErrEngineShutdown
269269
default:
@@ -313,7 +313,7 @@ func (el *eventloop) execCmd(itf interface{}) (err error) {
313313
314314
switch cmd.typ {
315315
case asyncCmdClose:
316-
return el.closeConn(c, nil)
316+
return el.close(c, nil)
317317
case asyncCmdWake:
318318
return el.wake(c)
319319
case asyncCmdWrite:

reactor_default_bsd.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ func (el *eventloop) activateSubReactor() error {
5656
if c := el.connections.getConn(fd); c != nil {
5757
switch filter {
5858
case netpoll.EVFilterSock:
59-
err = el.closeConn(c, unix.ECONNRESET)
59+
err = el.close(c, unix.ECONNRESET)
6060
case netpoll.EVFilterWrite:
6161
if !c.outboundBuffer.IsEmpty() {
6262
err = el.write(c)
@@ -90,7 +90,7 @@ func (el *eventloop) run() error {
9090
if c := el.connections.getConn(fd); c != nil {
9191
switch filter {
9292
case netpoll.EVFilterSock:
93-
err = el.closeConn(c, unix.ECONNRESET)
93+
err = el.close(c, unix.ECONNRESET)
9494
case netpoll.EVFilterWrite:
9595
if !c.outboundBuffer.IsEmpty() {
9696
err = el.write(c)

0 commit comments

Comments
 (0)