Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

修改promise内存布局,提高了安全性,减少了内存占用 #145

Closed
wants to merge 2 commits into from

Conversation

lee157953
Copy link

修改promise内存布局,提高了安全性,减少了内存占用

@duke-git
Copy link
Owner

@lee157953 赞,但是运行单测任务失败。参考下图:
image

单测失败修复
@lee157953
Copy link
Author

已修复

@duke-git
Copy link
Owner

已修复

测试任务还是失败。
image

@lee157953
Copy link
Author

已修复

测试任务还是失败。 image

方便告知哪里错误吗

@duke-git
Copy link
Owner

已修复

测试任务还是失败。 image

方便告知哪里错误吗

点击任务日志,只能看到如下图:,你可以在promise目录下执行:go test promise.go promise_test.go 看看能不能通过。如果通过,可能是github action的问题。

image

@lee157953
Copy link
Author

go test promise.go promise_test.go

已经通过

@duke-git
Copy link
Owner

go test promise.go promise_test.go

已经通过

单侧任务报以下错误:
image

@duke-git
Copy link
Owner

go test promise.go promise_test.go

已经通过

你用的go版本是不是1.19, lancet的go版本是1.18,里面没有sync/atomic这个包。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants