thirdpartymirrors/RingRacers
1
0
Fork 0
mirror of https://github.com/KartKrewDev/RingRacers.git synced 2025-10-30 08:01:28 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge branch 'charsel-softlock' into 'master'

Browse source 
M_CharacterSelectInit: Fix a heaping of bugs

See merge request KartKrew/Kart!1273
This commit is contained in:
Oni 2023-06-06 20:45:28 +00:00
commit e0e09073a7
1 changed files with 42 additions and 58 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

100
src/menus/play-char-select.c
View file

@ -7,6 +7,8 @@
#include "../k_grandprix.h" // K_CanChangeRules
#include "../m_cond.h" // Condition Sets
//#define CHARSELECT_DEVICEDEBUG
menuitem_t PLAY_CharSelect[] =
{
{IT_NOTHING, NULL, NULL, NULL, {NULL}, 0, 0},
@ -317,6 +319,9 @@ static void M_SetupProfileGridPos(setup_player_t *p)
INT32 i = R_SkinAvailable(pr->skinname);
INT32 alt = 0; // Hey it's my character's name!
if (i == -1)
i = 0;
// While we're here, read follower values.
p->followern = K_FollowerAvailable(pr->follower);
@ -349,6 +354,9 @@ static void M_SetupMidGameGridPos(setup_player_t *p, UINT8 num)
INT32 i = R_SkinAvailable(cv_skin[num].zstring);
INT32 alt = 0; // Hey it's my character's name!
if (i == -1)
i = 0;
// While we're here, read follower values.
p->followern = cv_follower[num].value;
p->followercolor = cv_followercolor[num].value;
@ -377,20 +385,6 @@ void M_CharacterSelectInit(void)
UINT8 i, j;
setup_maxpage = 0;
// While we're editing profiles, don't unset the devices for p1
if (gamestate == GS_MENU)
{
for (i = 0; i < MAXSPLITSCREENPLAYERS; i++)
{
// Un-set devices for all players if not editing profile
if (!optionsmenu.profile)
{
G_SetDeviceForPlayer(i, -1);
CONS_Printf("M_CharacterSelectInit: Device for %d set to %d\n", i, -1);
}
}
}
memset(setup_chargrid, -1, sizeof(setup_chargrid));
for (i = 0; i < 9; i++)
{
@ -404,8 +398,6 @@ void M_CharacterSelectInit(void)
memset(setup_explosions, 0, sizeof(setup_explosions));
setup_animcounter = 0;
UINT32 localskinhash[MAXSPLITSCREENPLAYERS];
for (i = 0; i < MAXSPLITSCREENPLAYERS; i++)
{
// Default to no follower / match colour.
@ -413,12 +405,35 @@ void M_CharacterSelectInit(void)
setup_player[i].followercategory = -1;
setup_player[i].followercolor = FOLLOWERCOLOR_MATCH;
// Set default selected profile to the last used profile for each player:
// (Make sure we don't overshoot it somehow if we deleted profiles or whatnot)
setup_player[i].profilen = min(cv_lastprofile[i].value, PR_GetNumProfiles());
// If we're on prpfile select, skip straight to CSSTEP_CHARS
// do the same if we're midgame, but make sure to consider splitscreen properly.
if (optionsmenu.profile && i == 0)
{
setup_player[i].profilen = optionsmenu.profilen;
//PR_ApplyProfileLight(setup_player[i].profilen, 0);
M_SetupProfileGridPos(&setup_player[i]);
setup_player[i].mdepth = CSSTEP_CHARS;
}
else
{
// Set default selected profile to the last used profile for each player:
// (Make sure we don't overshoot it somehow if we deleted profiles or whatnot)
setup_player[i].profilen = min(cv_lastprofile[i].value, PR_GetNumProfiles());
// Assistant for comparisons.
localskinhash[i] = quickncasehash(cv_skin[i].string, SKINNAMESIZE);
if (gamestate != GS_MENU && i <= splitscreen)
{
M_SetupMidGameGridPos(&setup_player[i], i);
setup_player[i].mdepth = CSSTEP_CHARS;
}
else
{
// Un-set devices
G_SetDeviceForPlayer(i, -1);
#ifdef CHARSELECT_DEVICEDEBUG
CONS_Printf("M_CharacterSelectInit: Device for %d set to %d\n", i, -1);
#endif
}
}
}
for (i = 0; i < numskins; i++)
@ -438,43 +453,6 @@ void M_CharacterSelectInit(void)
setup_maxpage = max(setup_maxpage, setup_chargrid[x][y].numskins-1);
}
for (j = 0; j < MAXSPLITSCREENPLAYERS; j++)
{
// See also R_SkinAvailable
if (localskinhash[j] != skins[i].namehash)
continue;
if (!stricmp(cv_skin[j].string, skins[i].name))
continue;
{
setup_player[j].gridx = x;
setup_player[j].gridy = y;
setup_player[j].color = skins[i].prefcolor;
// If we're on prpfile select, skip straight to CSSTEP_CHARS
// do the same if we're midgame, but make sure to consider splitscreen properly.
if ((optionsmenu.profile && j == 0) || (gamestate != GS_MENU && j <= splitscreen))
{
if (optionsmenu.profile) // In menu, setting up profile character/follower
{
setup_player[j].profilen = optionsmenu.profilen;
PR_ApplyProfileLight(setup_player[j].profilen, 0);
M_SetupProfileGridPos(&setup_player[j]);
}
else // gamestate != GS_MENU, in that case, assume this is whatever profile we chose to play with.
{
setup_player[j].profilen = cv_lastprofile[j].value;
M_SetupMidGameGridPos(&setup_player[j], j);
}
// Don't reapply the profile here, it was already applied.
setup_player[j].mdepth = CSSTEP_CHARS;
}
}
}
}
setup_numfollowercategories = 0;
@ -629,13 +607,17 @@ static boolean M_HandlePressStart(setup_player_t *p, UINT8 num)
}
G_SetDeviceForPlayer(num, device);
#ifdef CHARSELECT_DEVICEDEBUG
CONS_Printf("M_HandlePressStart: Device for %d set to %d\n", num, device);
#endif
for (j = num + 1; j < MAXSPLITSCREENPLAYERS; j++)
{
// Un-set devices for other players.
G_SetDeviceForPlayer(j, -1);
#ifdef CHARSELECT_DEVICEDEBUG
CONS_Printf("M_HandlePressStart: Device for %d set to %d\n", j, -1);
#endif
}
//setup_numplayers++;
@ -703,7 +685,9 @@ static boolean M_HandleCSelectProfile(setup_player_t *p, UINT8 num)
}
G_SetDeviceForPlayer(num, -1);
#ifdef CHARSELECT_DEVICEDEBUG
CONS_Printf("M_HandleCSelectProfile: Device for %d set to %d\n", num, -1);
#endif
return true;
}