-
Notifications
You must be signed in to change notification settings - Fork 305
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
feat: add sort module to stdlib #416
base: master
Are you sure you want to change the base?
Conversation
Thanks for the PR. Can you please add a well formatted and then code generate using |
@geseq Any updates on this? :) |
|
@geseq hello, I updated my code, please help to review |
Inline the swap function to reduce function calls |
37d9d04
to
3437f01
Compare
85e1629
to
ff637b2
Compare
hi @HuangHongkai, I am afraid some of the unit test cases in your commit won't work. would you like to try the sorting code below and request another PR? I believe it should work. let me know if any questions :)
|
related issue: #398
sort module source code