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

[RFC] About the version 5.x #150

Open
ZLY201 opened this issue Jan 16, 2023 · 0 comments · Fixed by #169
Open

[RFC] About the version 5.x #150

ZLY201 opened this issue Jan 16, 2023 · 0 comments · Fixed by #169
Assignees
Labels
enhancement New feature or request

Comments

@ZLY201
Copy link
Member

ZLY201 commented Jan 16, 2023

In previous versions, I relied too much on C++ STL.

In fact, many designs in C++ STL are not suitable for Javascript, especially in the function name specification.

This RFC (Request for Comment) outlines the all updates to js-sdsl which will be done in 2023.

  • Modify the names of all apis to conform to the specification in javascript.
  • Organize the code of the red-black tree or perform simple refactoring to make its code structure clearer. perf: optimize tree container #159
  • If possible further optimize hash table performance, at least as a good polyfill.
  • Improve performance of iterators for sequential containers, especially Deque.
  • Add algorithm and awesome module.
  • Change oop to functional programming. [Refused]
  • Change ESM building target to ES6. ESM build has classes downleveled to functions, but CJS build does not #158

Although this is a RFC for 5.x, I think some works could be done in version 4.x.

@ZLY201 ZLY201 added the enhancement New feature or request label Jan 16, 2023
@ZLY201 ZLY201 self-assigned this Jan 16, 2023
@ZLY201 ZLY201 linked a pull request May 14, 2023 that will close this issue
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

1 participant