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

use maps instead of tst #42

Merged
merged 13 commits into from
Jun 18, 2020
Merged

use maps instead of tst #42

merged 13 commits into from
Jun 18, 2020

Conversation

abahmed
Copy link
Member

@abahmed abahmed commented Jun 17, 2020

Fixes #41

Changes proposed in this pull request:

  • Use maps instead of TST

@abahmed abahmed requested a review from muhammednagy June 17, 2020 16:15
@abahmed abahmed added the enhancement New feature or request label Jun 17, 2020
@abahmed abahmed requested a review from abdallahokasha June 17, 2020 19:58
@abahmed abahmed added chore Improvements without changing the functionality and removed chore Improvements without changing the functionality labels Jun 17, 2020
@muhammednagy muhammednagy added enhancement New feature or request and removed enhancement New feature or request labels Jun 18, 2020
@muhammednagy muhammednagy merged commit 09ddb25 into master Jun 18, 2020
@abahmed abahmed deleted the use-maps-instead-of-tst branch June 19, 2020 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muhammednagy muhammednagy muhammednagy approved these changes

@abdallahokasha abdallahokasha Awaiting requested review from abdallahokasha

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Is a Ternary Search Tree really better?
2 participants
@abahmed @muhammednagy