work only process one task， so range for task channel is not necessary.

[kowloon-1997]

for fn := range w.task { if fn == nil { return } fn() if ok := w.pool.revertWorker(w); !ok { return } }

1. 当调用 Submit() 时，通过 retrieveWorker() 获取一个 worker
2. worker 执行完任务后，通过 revertWorker() 尝试将自己放回池中
3. 如果 revertWorker() 返回 true (ok=true)，worker 会继续监听新的任务
4. 但是这个 worker 已经从池中分离出来了（因为之前的 retrieveWorker()），其他的 Submit() 调用无法获取到这个 worker
5. 此时w.task会一直等待，造成内存泄漏

解决思路：
当前一个work只能处理一个task，而且需要让work回收复用成功，否则丢弃work。

`
fn := <-w.task
fn()

w.pool.revertWorker(w)
`

[panjf2000]

3. 但是这个 worker 已经从池中分离出来了（因为之前的 retrieveWorker()），其他的 Submit() 调用无法获取到这个 worker
4. 此时w.task会一直等待，造成内存泄漏

这个 worker 的实例放回到 pool，它就能被新的 retrieveWorker() 调用取到，然后它的 goroutine 还要继续运行，等待新的任务，这里本来就是这么设计的。建议再仔细看一下源码，你这个地方理解错了。

[kowloon-1997]

是的，理解错了，返回true说明work已经成功的被回收了。