zinx icon indicating copy to clipboard operation
zinx copied to clipboard
verified publisher icon aceld

异步问题

Open xajq opened this issue 2 years ago • 3 comments

async_op_result.go if aor.hasReturnedObj == 1 { aor.doComplete() } 改为 if atomic.LoadInt32(&aor.hasReturnedObj) == 1 { aor.doComplete() } 会不会更好

xajq avatar Sep 08 '23 09:09 xajq

因为前面的判断逻辑做了CAS了，所以这里其实用不用atomic都是可以的，不过改成atomic让语法统一是个不错的建议，欢迎pr~

hcraM41 avatar Sep 13 '23 09:09 hcraM41

已调整~

hcraM41 avatar Sep 20 '23 08:09 hcraM41

已调整~

感谢!!

xajq avatar Nov 29 '23 08:11 xajq