diff --git a/src/d_netcmd.c b/src/d_netcmd.c index 2f8bb0c5c..b55441b2d 100644 --- a/src/d_netcmd.c +++ b/src/d_netcmd.c @@ -6214,13 +6214,15 @@ static void Follower2_OnChange(void) } - INT32 num = R_FollowerAvailable(str); - char set[10]; - if (num == -1) // that's an error. - CONS_Alert(CONS_WARNING, M_GetText("Follower '%s' not found\n"), str); + { + INT32 num = R_FollowerAvailable(str); + char set[10]; + if (num == -1) // that's an error. + CONS_Alert(CONS_WARNING, M_GetText("Follower '%s' not found\n"), str); - sprintf(set, "%d", num); - CV_StealthSet(&cv_follower2, set); // set it to a number. It's easier for us to send later :) + sprintf(set, "%d", num); + CV_StealthSet(&cv_follower2, set); // set it to a number. It's easier for us to send later :) + } } SendNameAndColor2(); } @@ -6257,13 +6259,15 @@ static void Follower3_OnChange(void) return; } - INT32 num = R_FollowerAvailable(str); - char set[10]; - if (num == -1) // that's an error. - CONS_Alert(CONS_WARNING, M_GetText("Follower '%s' not found\n"), str); + { + INT32 num = R_FollowerAvailable(str); + char set[10]; + if (num == -1) // that's an error. + CONS_Alert(CONS_WARNING, M_GetText("Follower '%s' not found\n"), str); - sprintf(set, "%d", num); - CV_StealthSet(&cv_follower3, set); // set it to a number. It's easier for us to send later :) + sprintf(set, "%d", num); + CV_StealthSet(&cv_follower3, set); // set it to a number. It's easier for us to send later :) + } } SendNameAndColor3(); } @@ -6300,13 +6304,15 @@ static void Follower4_OnChange(void) return; } - INT32 num = R_FollowerAvailable(str); - char set[10]; - if (num == -1) // that's an error. - CONS_Alert(CONS_WARNING, M_GetText("Follower '%s' not found\n"), str); + { + INT32 num = R_FollowerAvailable(str); + char set[10]; + if (num == -1) // that's an error. + CONS_Alert(CONS_WARNING, M_GetText("Follower '%s' not found\n"), str); - sprintf(set, "%d", num); - CV_StealthSet(&cv_follower4, set); // set it to a number. It's easier for us to send later :) + sprintf(set, "%d", num); + CV_StealthSet(&cv_follower4, set); // set it to a number. It's easier for us to send later :) + } } SendNameAndColor4(); } diff --git a/src/dehacked.c b/src/dehacked.c index 2915fef2f..2111bf9ec 100644 --- a/src/dehacked.c +++ b/src/dehacked.c @@ -908,7 +908,7 @@ if (followers[numfollowers].field < threshold) \ FALLBACK(bubblescale, "BUBBLESCALE", 0, 0); // No negative scale // Special case for color I suppose - if (followers[numfollowers].defaultcolor < 0 || followers[numfollowers].defaultcolor > MAXSKINCOLORS-1) + if (followers[numfollowers].defaultcolor > MAXSKINCOLORS-1) { followers[numfollowers].defaultcolor = 1; deh_warning("Follower \'%s\': Value for 'color' should be between 1 and %d.\n", dname, MAXSKINCOLORS-1); diff --git a/src/k_pathfind.c b/src/k_pathfind.c index ddea477c8..316eab418 100644 --- a/src/k_pathfind.c +++ b/src/k_pathfind.c @@ -484,24 +484,24 @@ boolean K_PathfindAStar(path_t *const path, pathfindsetup_t *const pathfindsetup // Need to update pointers in closedset, openset, and node "camefrom" if nodesarray moved. if (nodesarray != nodesarrayrealloc) { - size_t i = 0U; + size_t j = 0U; size_t arrayindex = 0U; - for (i = 0U; i < closedsetcount; i++) + for (j = 0U; j < closedsetcount; j++) { - arrayindex = closedset[i] - nodesarray; - closedset[i] = &nodesarrayrealloc[arrayindex]; + arrayindex = closedset[j] - nodesarray; + closedset[j] = &nodesarrayrealloc[arrayindex]; } - for (i = 0U; i < openset.count; i++) + for (j = 0U; j < openset.count; j++) { - arrayindex = ((pathfindnode_t *)(openset.array[i].data)) - nodesarray; - openset.array[i].data = &nodesarrayrealloc[arrayindex]; + arrayindex = ((pathfindnode_t *)(openset.array[j].data)) - nodesarray; + openset.array[j].data = &nodesarrayrealloc[arrayindex]; } - for (i = 0U; i < nodesarraycount; i++) + for (j = 0U; j < nodesarraycount; j++) { - if (nodesarrayrealloc[i].camefrom != NULL) + if (nodesarrayrealloc[j].camefrom != NULL) { - arrayindex = nodesarrayrealloc[i].camefrom - nodesarray; - nodesarrayrealloc[i].camefrom = &nodesarrayrealloc[arrayindex]; + arrayindex = nodesarrayrealloc[j].camefrom - nodesarray; + nodesarrayrealloc[j].camefrom = &nodesarrayrealloc[arrayindex]; } } diff --git a/src/p_mobj.c b/src/p_mobj.c index 3bda4e05c..44a24242d 100644 --- a/src/p_mobj.c +++ b/src/p_mobj.c @@ -11797,8 +11797,8 @@ void P_RespawnSpecials(void) ss->sector->ceilingheight) - (mthing->options >> ZSHIFT) * FRACUNIT; if (mthing->options & MTF_AMBUSH && (i == MT_RING || i == MT_REDTEAMRING || i == MT_BLUETEAMRING || i == MT_COIN || P_WeaponOrPanel(i))) - z -= 24*FRACUNIT; - z -= mobjinfo[i].height; // Don't forget the height! + z -= 24 * mapobjectscale; + z -= FixedMul(mobjinfo[i].height, mapobjectscale); // Don't forget the height! } else { @@ -11809,7 +11809,7 @@ void P_RespawnSpecials(void) ss->sector->floorheight) + (mthing->options >> ZSHIFT) * FRACUNIT; if (mthing->options & MTF_AMBUSH && (i == MT_RING || i == MT_REDTEAMRING || i == MT_BLUETEAMRING || i == MT_COIN || P_WeaponOrPanel(i))) - z += 24*FRACUNIT; + z += 24 * mapobjectscale; } mo = P_SpawnMobj(x, y, z, i); diff --git a/src/p_user.c b/src/p_user.c index fd5691601..e826d7cd1 100644 --- a/src/p_user.c +++ b/src/p_user.c @@ -8492,9 +8492,11 @@ static void P_HandleFollower(player_t *player) // finally, add a cool floating effect to the z height. // not stolen from k_kart I swear!! - const fixed_t pi = (22<>ANGLETOFINESHIFT) & FINEMASK); - sz += FixedMul(player->mo->scale, sine)*P_MobjFlip(player->mo); + { + const fixed_t pi = (22<>ANGLETOFINESHIFT) & FINEMASK); + sz += FixedMul(player->mo->scale, sine)*P_MobjFlip(player->mo); + } // Set follower colour