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

Use single list as waiting queue #3712

Merged
merged 5 commits into from
Mar 9, 2022

Conversation

ranisalt
Copy link
Member

@ranisalt ranisalt commented Oct 2, 2021

Pull Request Prelude

Changes Proposed

Replaces the pair of queues (priority waitlist + standard waitlist) with a single list, keeping a token to the boundary. Reduces code complexity for calculating the length and finding a new slot.

Issues addressed:
None

@MillhioreBT
Copy link
Contributor

OMG I love how you write code in C++ 🤗

Copy link
Contributor

@EPuncker EPuncker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm stuck on this forever:

2021-10-11 18_14_44-Tibia

maxPlayers = 300

@ranisalt ranisalt changed the title Use single list as waiting queue Draft: Use single list as waiting queue Oct 13, 2021
@EPuncker EPuncker marked this pull request as draft October 23, 2021 05:22
@ranisalt ranisalt marked this pull request as ready for review February 16, 2022 00:06
@ranisalt
Copy link
Member Author

Fixed the infinite loading issue

EPuncker
EPuncker previously approved these changes Feb 16, 2022
@ranisalt ranisalt changed the title Draft: Use single list as waiting queue Use single list as waiting queue Feb 16, 2022
DSpeichert
DSpeichert previously approved these changes Feb 16, 2022
EPuncker
EPuncker previously approved these changes Feb 17, 2022
@EPuncker EPuncker merged commit 1df7090 into otland:master Mar 9, 2022
@ranisalt ranisalt deleted the waitlist-single-queue branch March 9, 2022 23:34
Codinablack pushed a commit to Codinablack/forgottenserver that referenced this pull request Apr 5, 2022
EPuncker pushed a commit to EPuncker/forgottenserver that referenced this pull request May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants