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.

Sign up for GitHub

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

Jump to bottom

algorithm/lrucache测试过于简陋, 且有bug #251

Open
eren1017 opened this issue Oct 11, 2024 · 1 comment
Open

algorithm/lrucache测试过于简陋, 且有bug #251

eren1017 opened this issue Oct 11, 2024 · 1 comment

Comments

@eren1017
Copy link

在Put方法中更新了length, 在Delete方法中却没有更新length
另外方法moveToHead含义错误, 代码中明明是move to tail
@duke-git
Copy link
Owner

在Put方法中更新了length, 在Delete方法中却没有更新length 另外方法moveToHead含义错误, 代码中明明是move to tail

后续会优化修复

aki-colt added a commit to aki-colt/lancet that referenced this issue Oct 15, 2024
@aki-colt
fix(algorithm): fix bug of lrucache in issue duke-git#251
88af35a
duke-git pushed a commit that referenced this issue Oct 16, 2024
@aki-colt
fix(algorithm): fix bug of lrucache in issue #251 (#254)
a254ebd
duke-git pushed a commit that referenced this issue Oct 18, 2024
@aki-colt
feat(netutil): add async http request for http client (#256)
1671f78 
* fix(algorithm): fix bug of lrucache in issue #251

* feat(netutil): add async send request for HttpClient
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@duke-git @eren1017