Skip to content

Re-implementation of Queue #80

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
Merged
merged 7 commits into from
Oct 13, 2019
Merged

Re-implementation of Queue #80

merged 7 commits into from
Oct 13, 2019

Conversation

ashokdey
Copy link
Member

👻 What's Inside?

Changes

  • The new Queue Implementation has Singly Linked List under the hood
  • The API has changed and is more close to the standard Queue Data Structure
  • List of methods
    • enqueue(data)
    • dequeue()
    • peek()
    • length()
  • Fixes all the test cases to use the new APIs
  • Updated the code of Weave a Queue to use new APIs

🎉

@ashokdey ashokdey requested a review from TheSTL October 13, 2019 08:24
@ashokdey ashokdey self-assigned this Oct 13, 2019
@TheSTL TheSTL merged commit 5bec3a5 into master Oct 13, 2019
@ashokdey ashokdey deleted the queue branch October 13, 2019 18:15
@TheSTL TheSTL mentioned this pull request Oct 14, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TheSTL TheSTL Awaiting requested review from TheSTL

Assignees

@ashokdey ashokdey

Labels
enhancement New feature or request new-data-structure strict-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ashokdey @TheSTL