-
-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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 2-3-4-tree implment #1366
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix clang-tidy
warnings.
Code is not up to the repository standards.
Please read them carefully and follow them.
|
hi Panquesito7, I have fix most of the warnings. But replacing fprintf() with std::cout and manging node with smart pointer is a little pain. Any suggestions? |
You can run GitPod on the web for more detailed errors. 🙂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Well done!
Add namespaces Co-authored-by: David Leal <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work, I like how you've made the code. 😄 👍
Code and documentation is pretty good and refined; LGTM. 🙂
Thank you for your credit and your help.😄 |
hi @ayaankhan98 , can you please review the code? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great work! Thanks for your contribution @fedom
thank you:) |
* feat: add 2-3-4 tree implment * updating DIRECTORY.md * docs: fix format issue of tab&space * fix: fix code format issues * fix: convert printf() to std::cout * fix: fix some clang-tidy warnings * fix: fix clang-tidy warnings of memory owning * fix: remove use of std::make_unique which is not support by c++11 * docs: improve documents * fix: replace fprint with ofstream, and improve docs * docs: improve docs for including header file * docs: improve file doces * fix: convert item type to int64_t, convert node item count type to int8_t * refactor: Apply suggestions from code review Add namespaces Co-authored-by: David Leal <[email protected]> * docs: remove obsolete comments Co-authored-by: liuhuan <[email protected]> Co-authored-by: github-actions <${GITHUB_ACTOR}@users.noreply.github.com> Co-authored-by: David Leal <[email protected]>
Description of Change
feat: add 2-3-4-tree implment
Checklist
Notes: