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

Define and use constants for maximum number of players and weapons #96

Closed
SamVanheer opened this issue Dec 1, 2021 · 1 comment
Closed

Define and use constants for maximum number of players and weapons #96

SamVanheer opened this issue Dec 1, 2021 · 1 comment
Assignees
@SamVanheer
Labels
enhancement New feature or request

Comments

@SamVanheer
Copy link
Collaborator

SamVanheer commented Dec 1, 2021 •
edited
Loading

Define constants for the maximum number of players and weapons supported.
There is an existing MAX_PLAYERS constant used by hud code, this should be renamed first to MAX_PLAYERS_HUD since it's defined to be 64 instead of 32.

The maximum number of weapons is defined to be 32 (MAX_WEAPONS), but it's actually 64. Change this value.

The constant MAX_CLIENTS also defines the maximum number of players, replace it with MAX_PLAYERS.
@SamVanheer SamVanheer added the enhancement New feature or request label Dec 1, 2021
@SamVanheer SamVanheer self-assigned this Dec 1, 2021
@SamVanheer SamVanheer changed the title Define constants for maximum number of players and weapons Define and use constants for maximum number of players and weapons Dec 1, 2021
SamVanheer added a commit that referenced this issue Dec 1, 2021
@SamVanheer
Rename MAX_PLAYERS to MAX_PLAYERS_HUD
fd87712 
#96
SamVanheer added a commit that referenced this issue Dec 1, 2021
@SamVanheer
Use MAX_PLAYERS everywhere for maximum player count
565c27f 
#96
@SamVanheer
Copy link
Collaborator Author

Additional commits:
e01645e
b7b59d7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SamVanheer SamVanheer

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

No branches or pull requests
1 participant
@SamVanheer