re
/
ants
forked from mirror/ants
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

🐯 Refactor variable "release" with atomic operation

This commit is contained in:
Andy Pan 2019-01-26 15:29:05 +08:00
parent b091435432
commit f8d71bb276
2 changed files with 12 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
pool.go
View File

@ -46,7 +46,7 @@ type Pool struct {
workers []*Worker
// release is used to notice the pool to closed itself.
release bool
release int32
// lock for synchronous operation.
lock sync.Mutex
@ -70,7 +70,7 @@ func (p *Pool) periodicallyPurge() {
currentTime := time.Now()
p.lock.Lock()
idleWorkers := p.workers
if len(idleWorkers) == 0 && p.Running() == 0 && p.release {
if len(idleWorkers) == 0 && p.Running() == 0 && atomic.LoadInt32(&p.release) == 1 {
p.lock.Unlock()
return
}
@ -120,12 +120,11 @@ func NewTimingPool(size, expiry int) (*Pool, error) {
// Submit submits a task to this pool.
func (p *Pool) Submit(task f) error {
if worker := p.getWorker(); worker != nil {
worker.task <- task
return nil
} else {
if 1 == atomic.LoadInt32(&p.release) {
return ErrPoolClosed
}
p.getWorker().task <- task
return nil
}
// Running returns the number of the currently running goroutines.
@ -158,8 +157,8 @@ func (p *Pool) ReSize(size int) {
// Release Closes this pool.
func (p *Pool) Release() error {
p.once.Do(func() {
atomic.StoreInt32(&p.release, 1)
p.lock.Lock()
p.release = true
idleWorkers := p.workers
for i, w := range idleWorkers {
w.task <- nil
@ -191,10 +190,6 @@ func (p *Pool) getWorker() *Worker {
p.lock.Lock()
defer p.lock.Unlock()
if p.release {
return nil
}
idleWorkers := p.workers
n := len(idleWorkers) - 1
if n < 0 {

17
pool_func.go
View File

@ -46,7 +46,7 @@ type PoolWithFunc struct {
workers []*WorkerWithFunc
// release is used to notice the pool to closed itself.
release bool
release int32
// lock for synchronous operation.
lock sync.Mutex
@ -73,7 +73,7 @@ func (p *PoolWithFunc) periodicallyPurge() {
currentTime := time.Now()
p.lock.Lock()
idleWorkers := p.workers
if len(idleWorkers) == 0 && p.Running() == 0 && p.release {
if len(idleWorkers) == 0 && p.Running() == 0 && atomic.LoadInt32(&p.release) == 1 {
p.lock.Unlock()
return
}
@ -124,12 +124,11 @@ func NewTimingPoolWithFunc(size, expiry int, f pf) (*PoolWithFunc, error) {
// Serve submits a task to pool.
func (p *PoolWithFunc) Serve(args interface{}) error {
if worker := p.getWorker(); worker != nil {
worker.args <- args
return nil
} else {
if 1 == atomic.LoadInt32(&p.release) {
return ErrPoolClosed
}
p.getWorker().args <- args
return nil
}
// Running returns the number of the currently running goroutines.
@ -162,8 +161,8 @@ func (p *PoolWithFunc) ReSize(size int) {
// Release Closed this pool.
func (p *PoolWithFunc) Release() error {
p.once.Do(func() {
atomic.StoreInt32(&p.release, 1)
p.lock.Lock()
p.release = true
idleWorkers := p.workers
for i, w := range idleWorkers {
w.args <- nil
@ -195,10 +194,6 @@ func (p *PoolWithFunc) getWorker() *WorkerWithFunc {
p.lock.Lock()
defer p.lock.Unlock()
if p.release {
return nil
}
idleWorkers := p.workers
n := len(idleWorkers) - 1
if n < 0 {