Skip to content
Snippets Groups Projects

Change MAXSKINS from 256 to 255

Merged Zwip-Zwap Zapony requested to merge Zwip-Zwap_Zapony/SRB2:maxskins-255 into next

!1466 (merged) increased MAXSKINS from 32 to 256, with a comment saying that it mustn't be set higher than 256 as some pieces of code use an 8-bit integer (0 to 255, or 256 different values) for skin numbers.
However, some of those pieces reserve a value of -1 (255) to mean "no skin", leaving only 0 to 254, or 255 different values, free for skins to occupy. This merge request reduces MAXSKINS to 255 so that a 256th skin won't conflict with "no skin".

For changelog purposes, this should be bundled into the "Raise the skin limit" part.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
Please register or sign in to reply
Loading