Skip to content
Snippets Groups Projects
Commit 29580282 authored by sphere's avatar sphere
Browse files

Merge branch 'maxskins-255' into 'next'

Change MAXSKINS from 256 to 255

See merge request !2365
parents e0749737 56c61a80
No related branches found
No related tags found
2 merge requests!2365Change MAXSKINS from 256 to 255,!2355fix newer versions of mixerx
Pipeline #2699 passed
...@@ -244,12 +244,12 @@ extern char logfilename[1024]; ...@@ -244,12 +244,12 @@ extern char logfilename[1024];
#define MAXPLAYERNAME 21 #define MAXPLAYERNAME 21
#define PLAYERSMASK (MAXPLAYERS-1) #define PLAYERSMASK (MAXPLAYERS-1)
// Don't make MAXSKINS higher than 256, since skin numbers are used with an // Don't make MAXSKINS higher than 255, since skin numbers are used with an UINT8 in
// UINT8 in various parts of the codebase. If you do anyway, the data type // various parts of the codebase, and one number is reserved. If you do anyway,
// of those variables will have to be changed into at least an UINT16. // the data type of those variables will have to be changed into at least an UINT16.
// This change must affect code such as demo recording and playback, // This change must affect code such as demo recording and playback,
// and the structure of some networking packets and commands. // and the structure of some networking packets and commands.
#define MAXSKINS 256 #define MAXSKINS 255
#define MAXCHARACTERSLOTS (MAXSKINS * 3) // Should be higher than MAXSKINS. #define MAXCHARACTERSLOTS (MAXSKINS * 3) // Should be higher than MAXSKINS.
#define COLORRAMPSIZE 16 #define COLORRAMPSIZE 16
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment