etc/sm64coopdx
0
0
Fork 0
mirror of https://github.com/coop-deluxe/sm64coopdx.git synced 2024-11-21 19:45:10 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fixed more possible bugs pointed out by cppcheck

Browse source
This commit is contained in:
MysterD 2022-03-01 21:13:00 -08:00
parent 637f17a8ac
commit 07cbeb119c
25 changed files with 674 additions and 53 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

621
cppcheck.log Normal file
View file

@ -0,0 +1,621 @@
src/audio/internal.h:54:0: error: failed to expand 'US_FLOAT', Invalid ## usage when expanding 'US_FLOAT'. [preprocessorErrorDirective]
#define US_FLOAT(x) x ## f
^
src/engine/level_script.c:435:17: warning: Redundant assignment of 'behaviorArg' to itself. [selfAssignment]
behaviorArg = behaviorArg;
^
src/engine/math_util.c:164:23: portability: Casting from float * * to unsigned int * is not portable due to different binary data representations on different platforms. [invalidPointerCast]
register u32 *d = (u32 *) dest;
^
src/engine/math_util.c:165:23: portability: Casting from float * * to unsigned int * is not portable due to different binary data representations on different platforms. [invalidPointerCast]
register u32 *s = (u32 *) src;
^
src/game/hud.c:32:9: style: struct member 'PowerMeterHUD::unused' is never used. [unusedStructMember]
f32 unused;
^
src/game/hud.c:36:9: style: struct member 'UnusedHUDStruct::unused1' is never used. [unusedStructMember]
u32 unused1;
^
src/game/hud.c:37:9: style: struct member 'UnusedHUDStruct::unused2' is never used. [unusedStructMember]
u16 unused2;
^
src/game/hud.c:38:9: style: struct member 'UnusedHUDStruct::unused3' is never used. [unusedStructMember]
u16 unused3;
^
src/game/ingame_menu.c:1311:5: warning: Assignment of function parameter has no effect outside the function. Did you forget dereferencing it? [uselessAssignmentPtrArg]
linePos += textLengths[multiTextId].length;
^
src/game/ingame_menu.c:866:29: style: Assignment 'currentIndex[0]=maxIndex' is redundant with condition 'currentIndex[0]==maxIndex'. [duplicateConditionalAssign]
currentIndex[0] = maxIndex;
^
src/game/ingame_menu.c:864:29: note: Condition 'currentIndex[0]==maxIndex'
if (currentIndex[0] == maxIndex) {
^
src/game/ingame_menu.c:866:29: note: Assignment 'currentIndex[0]=maxIndex' is redundant
currentIndex[0] = maxIndex;
^
src/game/ingame_menu.c:785:17: style: Finding the same value in both branches of ternary operator is suspicious as the same code is executed regardless of the condition. [duplicateValueTernary]
gDPLoadBlock(gDisplayListHead++, G_TX_LOADTILE, 0, 0, 8 * 8 - 1, CALC_DXT(8, G_IM_SIZ_8b_BYTES));
^
src/game/ingame_menu.c:793:21: style: Finding the same value in both branches of ternary operator is suspicious as the same code is executed regardless of the condition. [duplicateValueTernary]
gDPLoadBlock(gDisplayListHead++, G_TX_LOADTILE, 0, 0, 8 * 8 - 1, CALC_DXT(8, G_IM_SIZ_8b_BYTES));
^
src/game/ingame_menu.c:1084:68: style: Finding the same expression on both sides of an operator is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateExpression]
sInterpolatedDialogScale = gDialogBoxScale - 2 / 2;
^
src/game/ingame_menu.c:1087:68: style: Finding the same expression on both sides of an operator is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateExpression]
sInterpolatedDialogScale = gDialogBoxScale + 2 / 2;
^
src/game/ingame_menu.c:1100:68: style: Finding the same expression on both sides of an operator is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateExpression]
sInterpolatedDialogScale = gDialogBoxScale - 2 / 2;
^
src/game/ingame_menu.c:1102:68: style: Finding the same expression on both sides of an operator is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateExpression]
sInterpolatedDialogScale = gDialogBoxScale + 2 / 2;
^
src/game/mario.c:1283:46: warning: Either the condition 'm->squishTimer<=16' is redundant or the array 'sSquishScaleOverTime[16]' is accessed at index -1, which is out of bounds. [negativeIndex]
1.0f - ((sSquishScaleOverTime[15 - m->squishTimer] * 0.6f) / 100.0f);
^
src/game/mario.c:1279:33: note: Assuming that condition 'm->squishTimer<=16' is not redundant
else if (m->squishTimer <= 16) {
^
src/game/mario.c:1283:46: note: Negative array index
1.0f - ((sSquishScaleOverTime[15 - m->squishTimer] * 0.6f) / 100.0f);
^
src/pc/debuglog.h:15:17: style: Obsolete function 'asctime' called. It is recommended to use 'strftime' instead. [asctimeCalled]
char* str = asctime(&ltime2);
^
src/pc/logfile.h:34:17: style: Obsolete function 'asctime' called. It is recommended to use 'strftime' instead. [asctimeCalled]
char* str = asctime(&ltime2);
^
src/game/mario_actions_moving.c:1243:51: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
play_sound(m->flags & MARIO_METAL_CAP ? SOUND_ACTION_METAL_BONK : SOUND_ACTION_BONK,
^
src/game/mario_actions_cutscene.c:1548:47: portability: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS]
m->fadeWarpOpacity = (-m->actionTimer << 3) + 0xF8;
^
src/game/mario_actions_cutscene.c:134:49: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
strY = (sDispCreditsEntry->unk02 & 0x20 ? 28 : 172) + (numLines == 1) * 16;
^
src/game/mario_actions_cutscene.c:612:55: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
speed = gPlayer1Controller->buttonDown & B_BUTTON ? 1 : 4;
^
src/game/mario_actions_cutscene.c:755:46: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
set_mario_action(m, m->actionArg & 1 ? ACT_STAR_DANCE_NO_EXIT : ACT_STAR_DANCE_EXIT,
^
src/game/mario_actions_cutscene.c:1505:39: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
m, m->flags & MARIO_METAL_CAP ? SOUND_ACTION_METAL_JUMP : SOUND_ACTION_TERRAIN_JUMP, 1);
^
src/game/mario_actions_cutscene.c:1658:68: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
m->hurtCounter += m->flags & MARIO_CAP_ON_HEAD ? 12 : 18;
^
src/game/mario_actions_cutscene.c:2796:50: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
(gCurrCreditsEntry->unk02 & 0x10 ? width : -width) * 56 / 100 + 640;
^
src/game/mario_actions_cutscene.c:2798:50: style: Suspicious calculation. Please use parentheses to clarify the code. The code ''a&b?c:d'' should be written as either ''(a&b)?c:d'' or ''a&(b?c:d)''. [clarifyCalculation]
(gCurrCreditsEntry->unk02 & 0x20 ? height : -height) * 66 / 100 + 480;
^
src/game/mario_actions_moving.c:702:28: error: Uninitialized variable: dWallAngle [uninitvar]
if (m->wall == NULL || dWallAngle <= -0x71C8 || dWallAngle >= 0x71C8) {
^
src/game/bettercamera.inc.h:374:20: style: Condition 'newcam_analogue' is always true [knownConditionTrueFalse]
} else if (newcam_analogue) {
^
src/game/bettercamera.inc.h:346:25: note: Assuming that condition 'newcam_analogue==1' is not redundant
if (newcam_analogue == 1) {
^
src/game/bettercamera.inc.h:374:20: note: Condition 'newcam_analogue' is always true
} else if (newcam_analogue) {
^
src/game/camera.c:4385:9: style: Redundant initialization for 'yaw'. The initialized value is overwritten before it is read. [redundantInitialization]
yaw = calculate_yaw(origin, pos);
^
src/game/camera.c:4371:13: note: yaw is initialized
s16 yaw = gCamera->nextYaw;
^
src/game/camera.c:4385:9: note: yaw is overwritten
yaw = calculate_yaw(origin, pos);
^
src/game/camera.c:11516:19: style: Redundant initialization for 'targetFoV'. The initialized value is overwritten before it is read. [redundantInitialization]
targetFoV = 45.f;
^
src/game/camera.c:11513:19: note: targetFoV is initialized
f32 targetFoV = sFOVState.fov;
^
src/game/camera.c:11516:19: note: targetFoV is overwritten
targetFoV = 45.f;
^
src/game/camera.c:11536:19: style: Redundant initialization for 'targetFoV'. The initialized value is overwritten before it is read. [redundantInitialization]
targetFoV = 60.f;
^
src/game/camera.c:11533:19: note: targetFoV is initialized
f32 targetFoV = sFOVState.fov;
^
src/game/camera.c:11536:19: note: targetFoV is overwritten
targetFoV = 60.f;
^
src/game/camera.c:5468:9: style: Finding variables 'distTimer' and 'angleTimer' that are assigned the same expression is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateAssignExpression]
f32 distTimer = sModeTransition.framesLeft;
^
src/game/camera.c:5469:9: note: Same expression used in consecutive assignments of 'distTimer' and 'angleTimer'.
s16 angleTimer = sModeTransition.framesLeft;
^
src/game/camera.c:5468:9: note: Same expression used in consecutive assignments of 'distTimer' and 'angleTimer'.
f32 distTimer = sModeTransition.framesLeft;
^
src/game/camera.c:7062:39: style: Finding the same expression on both sides of an operator is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateExpression]
} while ((src[j].index != -1) && (src[j].index != -1)); //! same comparison performed twice
^
src/game/bettercamera.inc.h:692:24: warning: Undefined behaviour: float (257.55) to integer conversion overflow. [floatConversionOverflow]
newcam_xlu = (newcam_coldist-150)*2.55f;
^
src/game/bettercamera.inc.h:690:24: note: Assuming that condition 'newcam_coldist<=250' is not redundant
if (newcam_coldist <= 250 && (newcam_coldist-150)*2.55f < 255) {
^
src/game/bettercamera.inc.h:692:24: note: float to integer conversion
newcam_xlu = (newcam_coldist-150)*2.55f;
^
src/game/mario_misc.c:96:5: error: Shifting 32-bit value by 32 bits is undefined behaviour [shiftTooManyBits]
DEFINE_PLAYER_COLOR(0xf9, 0xeb, 0x30, /**/ 0x7f, 0x20, 0x7a),
^
src/game/profiler.c:83:9: style: Assignment of function parameter has no effect outside the function. [uselessAssignmentArg]
clockStart = 319;
^
src/game/profiler.c:86:9: style: Assignment of function parameter has no effect outside the function. [uselessAssignmentArg]
clockEnd = 319;
^
src/game/object_list_processor.c:510:20: style: Local variable 'unused' shadows outer argument [shadowArgument]
UNUSED s32 unused;
^
src/game/object_list_processor.c:490:41: note: Shadowed declaration
void spawn_objects_from_info(UNUSED s32 unused, struct SpawnInfo *spawnInfo) {
^
src/game/object_list_processor.c:510:20: note: Shadow variable
UNUSED s32 unused;
^
src/game/object_list_processor.c:537:69: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned]
u16 playerIndex = (spawnInfo->behaviorArg & ~(1 << 31));
^
src/game/save_file.c:150:63: error: Subtracting pointers that point to different objects [comparePointers]
const uintptr_t ofs = (u8*)&gSaveBuffer.files[file][slot] - (u8*)&gSaveBuffer;
^
src/game/save_file.h:72:21: note: Variable declared here.
struct SaveFile files[NUM_SAVE_FILES][2];
^
src/game/save_file.c:150:32: note: Address of variable taken here.
const uintptr_t ofs = (u8*)&gSaveBuffer.files[file][slot] - (u8*)&gSaveBuffer;
^
src/game/save_file.c:25:26: note: Variable declared here.
extern struct SaveBuffer gSaveBuffer;
^
src/game/save_file.c:150:70: note: Address of variable taken here.
const uintptr_t ofs = (u8*)&gSaveBuffer.files[file][slot] - (u8*)&gSaveBuffer;
^
src/game/save_file.c:150:63: note: Subtracting pointers that point to different objects
const uintptr_t ofs = (u8*)&gSaveBuffer.files[file][slot] - (u8*)&gSaveBuffer;
^
src/game/save_file.c:165:60: error: Subtracting pointers that point to different objects [comparePointers]
const uintptr_t ofs = (u8*)&gSaveBuffer.menuData[slot] - (u8*)&gSaveBuffer;
^
src/game/save_file.h:74:29: note: Variable declared here.
struct MainMenuSaveData menuData[2];
^
src/game/save_file.c:165:32: note: Address of variable taken here.
const uintptr_t ofs = (u8*)&gSaveBuffer.menuData[slot] - (u8*)&gSaveBuffer;
^
src/game/save_file.c:25:26: note: Variable declared here.
extern struct SaveBuffer gSaveBuffer;
^
src/game/save_file.c:165:67: note: Address of variable taken here.
const uintptr_t ofs = (u8*)&gSaveBuffer.menuData[slot] - (u8*)&gSaveBuffer;
^
src/game/save_file.c:165:60: note: Subtracting pointers that point to different objects
const uintptr_t ofs = (u8*)&gSaveBuffer.menuData[slot] - (u8*)&gSaveBuffer;
^
src/goddard/debug_utils.c:445:18: warning: Either the condition 'sNumRoutinesInStack>=(int)(sizeof(sRoutineNames)/sizeof(sRoutineNames[0]))' is redundant or the array 'sRoutineNames[64]' is accessed at index 64, which is out of bounds. [arrayIndexOutOfBoundsCond]
sRoutineNames[sNumRoutinesInStack] = NULL;
^
src/goddard/debug_utils.c:447:29: note: Assuming that condition 'sNumRoutinesInStack>=(int)(sizeof(sRoutineNames)/sizeof(sRoutineNames[0]))' is not redundant
if (sNumRoutinesInStack >= ARRAY_COUNT(sRoutineNames)) {
^
src/goddard/debug_utils.c:445:18: note: Array index out of bounds
sRoutineNames[sNumRoutinesInStack] = NULL;
^
src/goddard/draw_objects.c:482:13: style: Condition 'hasTextCoords' is always false [knownConditionTrueFalse]
if (hasTextCoords) {
^
src/goddard/draw_objects.c:448:21: note: Assignment 'hasTextCoords=0', assigned value is 0
hasTextCoords = FALSE;
^
src/goddard/draw_objects.c:482:13: note: Condition 'hasTextCoords' is always false
if (hasTextCoords) {
^
src/goddard/draw_objects.c:96:9: style: struct member 'Anonymous1::pad00' is never used. [unusedStructMember]
u32 pad00; // @ 801B9CE0
^
src/goddard/dynlist_proc.c:412:9: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
sprintf(buf, "N%d", DynIdAsInt(id));
^
src/goddard/dynlist_proc.c:549:9: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
sprintf(idbuf, "N%d", DynIdAsInt(id));
^
src/goddard/dynlist_proc.c:592:9: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
sprintf(sIntDynIdBuffer, "N%d", DynIdAsInt(id));
^
src/goddard/dynlist_proc.c:2923:13: style: Consecutive return, break, continue, goto or throw statements are unnecessary. The second statement can never be executed, and so should be removed. [duplicateBreak]
break; // lol
^
src/goddard/old_menu.c:73:9: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
sprintf(buf, "| %s %%x%d", group->name, (u32) (uintptr_t) group);
^
src/goddard/particles.c:250:49: style: Finding the same expression on both sides of an operator is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateExpression]
if (vtx1->header.type == OBJ_TYPE_PARTICLES && vtx1->header.type == OBJ_TYPE_PARTICLES) {
^
src/goddard/shape_helper.c:377:13: style: Consecutive return, break, continue, goto or throw statements are unnecessary. The second statement can never be executed, and so should be removed. [duplicateBreak]
continue; // unreachable
^
src/goddard/shape_helper.c:402:13: style: Consecutive return, break, continue, goto or throw statements are unnecessary. The second statement can never be executed, and so should be removed. [duplicateBreak]
continue; // unreachable + nonsensical
^
src/goddard/shape_helper.c:26:9: style: struct member 'UnkData::a' is never used. [unusedStructMember]
s32 a, b;
^
src/goddard/shape_helper.c:27:21: style: struct member 'UnkData::self' is never used. [unusedStructMember]
struct UnkData *self;
^
src/goddard/shape_helper.c:900:16: style: struct member 'Anonymous2::pad' is never used. [unusedStructMember]
u8 pad[0x40];
^
src/goddard/shape_helper.c:902:17: style: struct member 'Anonymous2::word44' is never used. [unusedStructMember]
s32 word44;
^
src/goddard/shape_helper.c:926:17: style: struct member 'Anonymous8::nv' is never used. [unusedStructMember]
f32 nv[3]; /* Guessing on the normals; they aren't used */
^
src/menu/intro_geo.c:106:21: style: Condition 'sp50==1' is always true [knownConditionTrueFalse]
} else if (sp50 == 1) {
^
src/menu/intro_geo.c:104:14: note: Assuming that condition 'sp50!=1' is not redundant
if (sp50 != 1) {
^
src/menu/intro_geo.c:106:21: note: Condition 'sp50==1' is always true
} else if (sp50 == 1) {
^
src/menu/intro_geo.c:149:21: style: Condition 'sp40==1' is always true [knownConditionTrueFalse]
} else if (sp40 == 1) {
^
src/menu/intro_geo.c:147:14: note: Assuming that condition 'sp40!=1' is not redundant
if (sp40 != 1) {
^
src/menu/intro_geo.c:149:21: note: Condition 'sp40==1' is always true
} else if (sp40 == 1) {
^
src/pc/configfile.c:231:9: error: Common realloc mistake: 'buffer' nulled but not freed upon failure [memleakOnRealloc]
buffer = realloc(buffer, bufferSize);
^
src/pc/discord/discordrpc.c:30:0: error: #error Unknown System [preprocessorErrorDirective]
# error Unknown System
^
src/menu/file_select.c:2272:49: error: Array 'textMarioAJustErased[1]' accessed at index 6, which is out of bounds. [arrayIndexOutOfBounds]
LANGUAGE_ARRAY(textMarioAJustErased)[MARIO_ERASED_VAR] = sSelectedFileIndex + 10;
^
src/menu/file_select.c:2307:38: style: The statement 'if (sStatusMessageID!=ERASE_MSG_PROMPT) sStatusMessageID=ERASE_MSG_PROMPT' is logically equivalent to 'sStatusMessageID=ERASE_MSG_PROMPT'. [duplicateConditionalAssign]
if (sStatusMessageID != ERASE_MSG_PROMPT) {
^
src/menu/file_select.c:2308:38: note: Assignment 'sStatusMessageID=ERASE_MSG_PROMPT'
sStatusMessageID = ERASE_MSG_PROMPT;
^
src/menu/file_select.c:2307:38: note: Condition 'sStatusMessageID!=ERASE_MSG_PROMPT' is redundant
if (sStatusMessageID != ERASE_MSG_PROMPT) {
^
src/pc/djui/djui_checkbox.c:36:6: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: checkbox. [nullPointerRedundantCheck]
*checkbox->value = !(*checkbox->value);
^
src/pc/djui/djui_checkbox.c:38:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_checkbox.c:35:37: note: Assignment 'checkbox=(struct DjuiCheckbox*)base', assigned value is 0
struct DjuiCheckbox* checkbox = (struct DjuiCheckbox*)base;
^
src/pc/djui/djui_checkbox.c:36:6: note: Null pointer dereference
*checkbox->value = !(*checkbox->value);
^
src/pc/djui/djui_checkbox.c:37:56: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: checkbox. [nullPointerRedundantCheck]
djui_base_set_visible(&checkbox->rectValue->base, *checkbox->value);
^
src/pc/djui/djui_checkbox.c:38:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_checkbox.c:35:37: note: Assignment 'checkbox=(struct DjuiCheckbox*)base', assigned value is 0
struct DjuiCheckbox* checkbox = (struct DjuiCheckbox*)base;
^
src/pc/djui/djui_checkbox.c:37:56: note: Null pointer dereference
djui_base_set_visible(&checkbox->rectValue->base, *checkbox->value);
^
src/goddard/renderer.c:2812:13: style: Assignment of function parameter has no effect outside the function. [uselessAssignmentArg]
fmt = 3;
^
src/goddard/renderer.c:2820:13: style: Assignment of function parameter has no effect outside the function. [uselessAssignmentArg]
size = 2;
^
src/goddard/renderer.c:934:25: style: Consecutive return, break, continue, goto or throw statements are unnecessary. The second statement can never be executed, and so should be removed. [duplicateBreak]
break; // needed to match
^
src/goddard/renderer.c:2188:5: style: Consecutive return, break, continue, goto or throw statements are unnecessary. The second statement can never be executed, and so should be removed. [duplicateBreak]
return; // @ 801A17A8
^
src/pc/djui/djui_hud_utils.c:128:23: warning: Logical conjunction always evaluates to false: c == '\n' && c == ' '. Are these conditions necessary? Did you intend to use || instead? Are the numbers correct? Are you comparing the correct variables? [incorrectLogicOperator]
if (c == '\n' && c == ' ') {
^
src/pc/djui/djui_inputbox.c:113:15: style: The unsigned expression 'i' will never be negative so it is either pointless or an error to check if it is. [unsignedLessThanZero]
if (i <= 0) { i = 0; break; }
^
src/goddard/renderer.c:83:32: style: struct member 'DynListBankInfo::list' is never used. [unusedStructMember]
/* 0x04 */ struct DynList *list;
^
src/pc/djui/djui_panel_host.c:93:13: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
snprintf(portString, 32, "%d", configHostPort);
^
src/pc/djui/djui_panel_join.c:119:9: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
snprintf(buffer, 256, "%s:%d", configJoinIp, configJoinPort);
^
src/pc/djui/djui_panel_host_settings.c:102:13: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
snprintf(limitString, 32, "%d", configAmountofPlayers);
^
src/pc/djui/djui_selectionbox.c:50:13: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: selectionbox. [nullPointerRedundantCheck]
f32 x = selectionbox->rect->base.elem.x;
^
src/pc/djui/djui_selectionbox.c:54:18: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_selectionbox.c:49:45: note: Assignment 'selectionbox=(struct DjuiSelectionbox*)base', assigned value is 0
struct DjuiSelectionbox* selectionbox = (struct DjuiSelectionbox*)base;
^
src/pc/djui/djui_selectionbox.c:50:13: note: Null pointer dereference
f32 x = selectionbox->rect->base.elem.x;
^
src/pc/fs/fs.c:225:16: style: Non-boolean value returned from function returning bool [returnNonBoolInBooleanFunction]
if (!file) return -1;
^
src/pc/djui/djui_slider.c:48:18: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: slider. [nullPointerRedundantCheck]
u32 min = slider->min;
^
src/pc/djui/djui_slider.c:57:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_slider.c:47:33: note: Assignment 'slider=(struct DjuiSlider*)base', assigned value is 0
struct DjuiSlider* slider = (struct DjuiSlider*)base;
^
src/pc/djui/djui_slider.c:48:18: note: Null pointer dereference
u32 min = slider->min;
^
src/pc/djui/djui_slider.c:49:18: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: slider. [nullPointerRedundantCheck]
u32 max = slider->max;
^
src/pc/djui/djui_slider.c:57:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_slider.c:47:33: note: Assignment 'slider=(struct DjuiSlider*)base', assigned value is 0
struct DjuiSlider* slider = (struct DjuiSlider*)base;
^
src/pc/djui/djui_slider.c:49:18: note: Null pointer dereference
u32 max = slider->max;
^
src/pc/djui/djui_slider.c:50:18: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: slider. [nullPointerRedundantCheck]
u32* value = slider->value;
^
src/pc/djui/djui_slider.c:57:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_slider.c:47:33: note: Assignment 'slider=(struct DjuiSlider*)base', assigned value is 0
struct DjuiSlider* slider = (struct DjuiSlider*)base;
^
src/pc/djui/djui_slider.c:50:18: note: Null pointer dereference
u32* value = slider->value;
^
src/pc/djui/djui_slider.c:51:13: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: slider. [nullPointerRedundantCheck]
f32 x = slider->rect->base.elem.x;
^
src/pc/djui/djui_slider.c:57:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_slider.c:47:33: note: Assignment 'slider=(struct DjuiSlider*)base', assigned value is 0
struct DjuiSlider* slider = (struct DjuiSlider*)base;
^
src/pc/djui/djui_slider.c:51:13: note: Null pointer dereference
f32 x = slider->rect->base.elem.x;
^
src/pc/djui/djui_slider.c:52:13: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: slider. [nullPointerRedundantCheck]
f32 w = slider->rect->base.elem.width;
^
src/pc/djui/djui_slider.c:57:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_slider.c:47:33: note: Assignment 'slider=(struct DjuiSlider*)base', assigned value is 0
struct DjuiSlider* slider = (struct DjuiSlider*)base;
^
src/pc/djui/djui_slider.c:52:13: note: Null pointer dereference
f32 w = slider->rect->base.elem.width;
^
src/pc/djui/djui_slider.c:85:18: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: slider. [nullPointerRedundantCheck]
int value = *slider->value;
^
src/pc/djui/djui_slider.c:90:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_slider.c:83:33: note: Assignment 'slider=(struct DjuiSlider*)base', assigned value is 0
struct DjuiSlider* slider = (struct DjuiSlider*)base;
^
src/pc/djui/djui_slider.c:85:18: note: Null pointer dereference
int value = *slider->value;
^
src/pc/djui/djui_slider.c:89:6: warning: Either the condition 'base!=(void*)0' is redundant or there is possible null pointer dereference: slider. [nullPointerRedundantCheck]
*slider->value = value;
^
src/pc/djui/djui_slider.c:90:14: note: Assuming that condition 'base!=(void*)0' is not redundant
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
^
src/pc/djui/djui_slider.c:83:33: note: Assignment 'slider=(struct DjuiSlider*)base', assigned value is 0
struct DjuiSlider* slider = (struct DjuiSlider*)base;
^
src/pc/djui/djui_slider.c:89:6: note: Null pointer dereference
*slider->value = value;
^
src/pc/fs/fs_packtype_zip.c:132:18: style: struct member 'lfh_s::version_required' is never used. [unusedStructMember]
uint16_t version_required;
^
src/pc/fs/fs_packtype_zip.c:135:18: style: struct member 'lfh_s::mod_time' is never used. [unusedStructMember]
uint16_t mod_time;
^
src/pc/fs/fs_packtype_zip.c:136:18: style: struct member 'lfh_s::mod_date' is never used. [unusedStructMember]
uint16_t mod_date;
^
src/pc/fs/fs_packtype_zip.c:167:18: style: struct member 'cdh_s::version_used' is never used. [unusedStructMember]
uint16_t version_used;
^
src/pc/fs/fs_packtype_zip.c:168:18: style: struct member 'cdh_s::version_required' is never used. [unusedStructMember]
uint16_t version_required;
^
src/pc/fs/fs_packtype_zip.c:171:18: style: struct member 'cdh_s::mod_time' is never used. [unusedStructMember]
uint16_t mod_time;
^
src/pc/fs/fs_packtype_zip.c:172:18: style: struct member 'cdh_s::mod_date' is never used. [unusedStructMember]
uint16_t mod_date;
^
src/pc/fs/fs_packtype_zip.c:179:18: style: struct member 'cdh_s::start_disk' is never used. [unusedStructMember]
uint16_t start_disk;
^
src/pc/gfx/gfx_opengl.c:635:74: style: Condition 'is_es' is always false [knownConditionTrueFalse]
sys_fatal("OpenGL 2.1+ is required.\nReported version: %s%d.%d", is_es ? "ES" : "", vmajor, vminor);
^
src/pc/gfx/gfx_opengl.c:634:37: note: Assuming that condition '!is_es' is not redundant
if (vmajor < 2 && vminor < 1 && !is_es)
^
src/pc/gfx/gfx_opengl.c:635:74: note: Condition 'is_es' is always false
sys_fatal("OpenGL 2.1+ is required.\nReported version: %s%d.%d", is_es ? "ES" : "", vmajor, vminor);
^
src/pc/gfx/gfx_pc.c:1878:11: style: Finding variables 'minX' and 'maxX' that are assigned the same expression is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateAssignExpression]
float minX = rsp.loaded_vertices[start_index].x;
^
src/pc/gfx/gfx_pc.c:1879:11: note: Same expression used in consecutive assignments of 'minX' and 'maxX'.
float maxX = rsp.loaded_vertices[start_index].x;
^
src/pc/gfx/gfx_pc.c:1878:11: note: Same expression used in consecutive assignments of 'minX' and 'maxX'.
float minX = rsp.loaded_vertices[start_index].x;
^
src/pc/gfx/gfx_pc.c:1880:11: style: Finding variables 'minY' and 'maxY' that are assigned the same expression is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateAssignExpression]
float minY = rsp.loaded_vertices[start_index].y;
^
src/pc/gfx/gfx_pc.c:1881:11: note: Same expression used in consecutive assignments of 'minY' and 'maxY'.
float maxY = rsp.loaded_vertices[start_index].y;
^
src/pc/gfx/gfx_pc.c:1880:11: note: Same expression used in consecutive assignments of 'minY' and 'maxY'.
float minY = rsp.loaded_vertices[start_index].y;
^
src/pc/gfx/gfx_pc.c:1883:11: style: Finding variables 'minU' and 'maxU' that are assigned the same expression is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateAssignExpression]
float minU = rsp.loaded_vertices[start_index].u;
^
src/pc/gfx/gfx_pc.c:1884:11: note: Same expression used in consecutive assignments of 'minU' and 'maxU'.
float maxU = rsp.loaded_vertices[start_index].u;
^
src/pc/gfx/gfx_pc.c:1883:11: note: Same expression used in consecutive assignments of 'minU' and 'maxU'.
float minU = rsp.loaded_vertices[start_index].u;
^
src/pc/gfx/gfx_pc.c:1885:11: style: Finding variables 'minV' and 'maxV' that are assigned the same expression is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [duplicateAssignExpression]
float minV = rsp.loaded_vertices[start_index].v;
^
src/pc/gfx/gfx_pc.c:1886:11: note: Same expression used in consecutive assignments of 'minV' and 'maxV'.
float maxV = rsp.loaded_vertices[start_index].v;
^
src/pc/gfx/gfx_pc.c:1885:11: note: Same expression used in consecutive assignments of 'minV' and 'maxV'.
float minV = rsp.loaded_vertices[start_index].v;
^
src/pc/lua/smlua_cobject.c:86:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("_get_field on invalid LOT '%u'", lot);
^
src/pc/lua/smlua_cobject.c:92:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("_get_field received a pointer not in allow list. '%u', '%llu", lot, (u64)pointer);
^
src/pc/lua/smlua_cobject.c:160:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("_set_field on invalid LOT '%u'", lot);
^
src/pc/lua/smlua_cobject.c:166:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("_set_field received a pointer not in allow list. '%u', '%llu", lot, (u64)pointer);
^
src/pc/lua/smlua_cobject.c:113:50: portability: Casting between unsigned char * and float * which have an incompatible binary data representation. [invalidPointerCast]
case LVT_F32: lua_pushnumber( L, *(f32*)p); break;
^
src/pc/lua/smlua_cobject.c:194:24: portability: Casting between unsigned char * and float * which have an incompatible binary data representation. [invalidPointerCast]
case LVT_F32: *(f32*)p = smlua_to_number(L, -1); break;
^
src/pc/lua/smlua_cobject.c:53:23: style: Finding the same expression on both sides of an operator is suspicious and might indicate a cut and paste or logic error. Please examine this code carefully to determine if it is correct. [knownConditionTrueFalse]
if (min > max || max < min) {
^
src/pc/lua/smlua_hooks.c:56:13: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("Failed to call the event_hook callback: %u, %s", hookType, lua_tostring(L, -1));
^
src/pc/lua/smlua_hooks.c:79:13: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("Failed to call the callback: %u, %s", hookType, lua_tostring(L, -1));
^
src/pc/lua/smlua_hooks.c:108:13: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("Failed to call the callback: %u, %s", hookType, lua_tostring(L, -1));
^
src/pc/lua/smlua_hooks.c:140:13: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("Failed to call the callback: %u, %s", hookType, lua_tostring(L, -1));
^
src/pc/lua/smlua_hooks.c:163:13: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("Failed to call the callback: %u, %s", hookType, lua_tostring(L, -1));
^
src/pc/lua/smlua_functions.c:15:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("improper param count: expected %u, received %u", expected, top);
^
src/pc/lua/smlua_functions.c:15:9: warning: %u in format string (no. 2) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("improper param count: expected %u, received %u", expected, top);
^
src/pc/lua/smlua_functions.c:25:9: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("improper param count: expected (%u - %u), received %u", min, max, top);
^
src/pc/lua/smlua_functions.c:25:9: warning: %u in format string (no. 2) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("improper param count: expected (%u - %u), received %u", min, max, top);
^
src/pc/lua/smlua_functions.c:25:9: warning: %u in format string (no. 3) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("improper param count: expected (%u - %u), received %u", min, max, top);
^
src/pc/lua/smlua_sync_table.c:176:12: warning: Redundant assignment of 'lntKey' to itself. [selfAssignment]
lntKey = lntKey;
^
src/pc/lua/smlua_obj_utils.c:54:13: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOG_LUA("Failed to call the callback: %u, %s", objSetupFunction, lua_tostring(L, -1));
^
src/pc/network/ban_list.c:17:9: error: Common realloc mistake: 'gBanAddresses' nulled but not freed upon failure [memleakOnRealloc]
gBanAddresses = realloc(gBanAddresses, sizeof(char*) * gBanCount);
^
src/pc/network/ban_list.c:19:9: error: Common realloc mistake: 'gBanPerm' nulled but not freed upon failure [memleakOnRealloc]
gBanPerm = realloc(gBanPerm, sizeof(bool) * gBanCount);
^
src/pc/network/discord/activity.c:26:9: style: Consecutive return, break, continue, goto or throw statements are unnecessary. The second statement can never be executed, and so should be removed. [duplicateBreak]
return;
^
src/pc/network/discord/lobby.c:40:5: warning: %u in format string (no. 1) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint]
LOGFILE_INFO(LFT_DISCORD, "Lobby type: %u", lobby->type);
^
src/pc/network/network_player.c:306:19: style: Expression is always false because 'else if' condition matches previous condition at line 304. [multiCondition]
} else if (matchingLocal && gNetworkPlayerLocal->currCourseNum != 0) {
^
src/pc/network/packets/packet_debug_sync.c:11:20: warning: Redundant assignment of 'behaviorId' to itself. [selfAssignment]
behaviorId = behaviorId; // suppress warning
^
src/pc/network/packets/packet_level_spawn_info.c:108:32: style: The unsigned expression 'spawnInfoDeletionCount' will never be negative so it is either pointless or an error to check if it is. [unsignedLessThanZero]
if (spawnInfoDeletionCount <= 0) { return; }
^
src/pc/network/socket/domain_res.c:19:15: style: Redundant initialization for 'host_name'. The initialized value is overwritten before it is read. [redundantInitialization]
host_name = configJoinIp;
^
src/pc/network/socket/domain_res.c:16:21: note: host_name is initialized
char *host_name = configJoinIp;
^
src/pc/network/socket/domain_res.c:19:15: note: host_name is overwritten
host_name = configJoinIp;
^
src/pc/network/socket/socket.c:89:9: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
snprintf(joinText, 63, "%s %d", configJoinIp, configJoinPort);
^

2
developer/cpp-check.sh
View file

@ -3,7 +3,7 @@
if [ $# -eq 0 ]; then
EXTRA=""
else
EXTRA="--enable=all --suppress=constStatement --suppress=duplicateCondition --suppress=redundantAssignment --suppress=variableScope --suppress=unusedVariable --suppress=unreadVariable"
EXTRA="--enable=all --suppress=constStatement --suppress=duplicateCondition --suppress=redundantAssignment --suppress=variableScope --suppress=unusedVariable --suppress=unreadVariable --suppress=shadowVariable"
fi
INCLUDES="-Iinclude -Isrc -I. -Ilib/lua/include"

2
include/sm64.h
View file

@ -167,7 +167,7 @@
#define ACT_FLAG_PAUSE_EXIT /* 0x08000000 */ (1 << 27)
#define ACT_FLAG_SWIMMING_OR_FLYING /* 0x10000000 */ (1 << 28)
#define ACT_FLAG_WATER_OR_TEXT /* 0x20000000 */ (1 << 29)
#define ACT_FLAG_THROWING /* 0x80000000 */ (1 << 31)
#define ACT_FLAG_THROWING /* 0x80000000 */ ((u32)1 << 31)
#define ACT_UNINITIALIZED 0x00000000 // (0x000)

2
lib/src/hardware.h
View file

@ -7,7 +7,7 @@
#define AI_LEN_REG 0x04500004
#define AI_CONTROL_REG 0x04500008
#define AI_STATUS_REG 0x0450000C
#define AI_STATUS_AI_FULL (1 << 31)
#define AI_STATUS_AI_FULL ((u32)1 << 31)
#define AI_STATUS_AI_BUSY (1 << 30)
#define AI_DACRATE_REG 0x04500010
#define AI_BITRATE_REG 0x04500014

7
src/audio/external.c
View file

@ -24,6 +24,7 @@
// files. We should really fix our naming to be less ambiguous...
#define MAX_BG_MUSIC_QUEUE_SIZE 6
#define SOUND_BANK_COUNT 12
#define SOUND_INDEX_COUNT 40
#define MAX_CHANNELS_PER_SOUND 1
#define SEQUENCE_NONE 0xFF
@ -359,7 +360,7 @@ struct ChannelVolumeScaleFade D_80360928[3][CHANNELS_MAX];
u8 sUsedChannelsForSoundBank[SOUND_BANK_COUNT];
u8 sCurrentSound[SOUND_BANK_COUNT][MAX_CHANNELS_PER_SOUND]; // index into gSoundBanks
// list item memory for D_803320A4 and D_803320B0
struct SoundCharacteristics gSoundBanks[SOUND_BANK_COUNT][40] = { 0 };
struct SoundCharacteristics gSoundBanks[SOUND_BANK_COUNT][SOUND_INDEX_COUNT] = { 0 };
u8 D_80363808[SOUND_BANK_COUNT];
u8 D_80363812;
static u8 sCapVolumeTo40;
@ -1073,7 +1074,7 @@ void update_game_sound(void) {
func_8031E16C(bankIndex);
for (j = 0; j < MAX_CHANNELS_PER_SOUND; j++) {
index = sCurrentSound[bankIndex][j];
if (index < 0xff && gSoundBanks[bankIndex][index].soundStatus != SOUND_STATUS_STOPPED) {
if (index < SOUND_INDEX_COUNT && gSoundBanks[bankIndex][index].soundStatus != SOUND_STATUS_STOPPED) {
soundStatus = gSoundBanks[bankIndex][index].soundBits & SOUNDARGS_MASK_STATUS;
soundId = (gSoundBanks[bankIndex][index].soundBits >> SOUNDARGS_SHIFT_SOUNDID);
gSoundBanks[bankIndex][index].soundStatus = soundStatus;
@ -1378,7 +1379,7 @@ void update_game_sound(void) {
}
// add custom pitch bend
if (gSoundBanks[bankIndex][index].customFreqScale != 0) {
if (index < SOUND_INDEX_COUNT && gSoundBanks[bankIndex][index].customFreqScale != 0) {
gSequencePlayers[SEQ_PLAYER_SFX].channels[channelIndex]->freqScale *= gSoundBanks[bankIndex][index].customFreqScale;
}
channelIndex++;

6
src/audio/seqplayer.c
View file

@ -1219,8 +1219,9 @@ void sequence_channel_process_script(struct SequenceChannel *seqChannel) {
break;
case 0xf8: // chan_loop; loop start, N iterations (or 256 if N = 0)
if (0 && state->depth >= 4) {
if (state->depth >= 4) {
eu_stubbed_printf_0("Audio:Track :Loops Macro Level Over Error!\n");
break;
}
state->remLoopIters[state->depth] = m64_read_u8(state);
#ifdef VERSION_EU
@ -1835,8 +1836,9 @@ void sequence_player_process_sequence(struct SequencePlayer *seqPlayer) {
break;
case 0xf8: // seq_loop; loop start, N iterations (or 256 if N = 0)
if (0 && state->depth >= 4) {
if (state->depth >= 4) {
eu_stubbed_printf_0("Macro Level Over Error!\n");
break;
}
state->remLoopIters[state->depth] = m64_read_u8(state);
#ifdef VERSION_EU

2
src/engine/level_script.c
View file

@ -444,7 +444,7 @@ static void level_cmd_init_mario(void) {
spawnInfo->activeAreaIndex = -1;
spawnInfo->areaIndex = 0;
spawnInfo->behaviorArg = (u32)i | (1 << 31);
spawnInfo->behaviorArg = (u32)i | ((u32)1 << 31);
spawnInfo->behaviorScript = behaviorScript;
spawnInfo->unk18 = unk18;
spawnInfo->next = NULL;

12
src/game/behaviors/amp.inc.c
View file

@ -47,15 +47,9 @@ static void check_amp_attack(void) {
obj_set_hitbox(o, &sAmpHitbox);
if (o->oInteractStatus & INT_STATUS_INTERACTED) {
// Unnecessary if statement, maybe caused by a macro for
// if (o->oInteractStatus & INT_STATUS_INTERACTED)
// o->oAction = X;
// ?
if (o->oInteractStatus & INT_STATUS_INTERACTED) {
// This function is used for both normal amps and homing amps,
// AMP_ACT_ATTACK_COOLDOWN == HOMING_AMP_ACT_ATTACK_COOLDOWN
o->oAction = AMP_ACT_ATTACK_COOLDOWN;
}
// This function is used for both normal amps and homing amps,
// AMP_ACT_ATTACK_COOLDOWN == HOMING_AMP_ACT_ATTACK_COOLDOWN
o->oAction = AMP_ACT_ATTACK_COOLDOWN;
// Clear interact status
o->oInteractStatus = 0;

4
src/game/behaviors/bowser.inc.c
View file

@ -875,7 +875,9 @@ void bowser_spawn_grand_star_key(void) {
network_send_spawn_objects(spawn_objects, models, 1);
}
}
gSecondCameraFocus->oAngleVelYaw = o->oAngleVelYaw;
if (gSecondCameraFocus != NULL) {
gSecondCameraFocus->oAngleVelYaw = o->oAngleVelYaw;
}
}
void bowser_fly_back_dead(void) {

3
src/game/behaviors/camera_lakitu.inc.c
View file

@ -68,8 +68,7 @@ static void camera_lakitu_intro_act_trigger_cutscene(void) {
//! These bounds are slightly smaller than the actual bridge bounds, allowing
// the RTA speedrunning method of lakitu skip
if (player->oPosX > -544.0f && player->oPosX < 545.0f && player->oPosY > 800.0f
&& player->oPosZ > -2000.0f && player->oPosZ < -177.0f
&& player->oPosZ < -177.0f) // always double check your conditions
&& player->oPosZ > -2000.0f && player->oPosZ < -177.0f)
{
if (should_start_or_continue_dialog(marioState, o) && set_mario_npc_dialog(&gMarioStates[0], 2, camera_lakitu_intro_act_trigger_cutscene_continue_dialog) == 1) {
o->oAction = CAMERA_LAKITU_INTRO_ACT_SPAWN_CLOUD;

2
src/game/behaviors/chuckya.inc.c
View file

@ -88,7 +88,7 @@ void chuckya_act_0(void) {
s32 sp28;
if (o->oTimer == 0)
o->oChuckyaUnkFC = 0;
angleToPlayer = obj_angle_to_object(o, player);
switch (sp28 = o->oSubAction) {
case 0:
o->oForwardVel = 0;

9
src/game/behaviors/jrb_ship.inc.c
View file

@ -65,11 +65,12 @@ void bhv_jrb_sliding_box_loop(void) {
s16 sp1E;
if (o->oJrbSlidingBoxUnkF4 == NULL) {
sp3C = cur_obj_nearest_object_with_behavior(bhvInSunkenShip3);
if (sp3C != NULL) // NULL check only for assignment, not for dereference?
if (sp3C != NULL) {
o->oJrbSlidingBoxUnkF4 = sp3C;
o->oParentRelativePosX = o->oPosX - sp3C->oPosX;
o->oParentRelativePosY = o->oPosY - sp3C->oPosY;
o->oParentRelativePosZ = o->oPosZ - sp3C->oPosZ;
o->oParentRelativePosX = o->oPosX - sp3C->oPosX;
o->oParentRelativePosY = o->oPosY - sp3C->oPosY;
o->oParentRelativePosZ = o->oPosZ - sp3C->oPosZ;
}
} else {
sp3C = o->oJrbSlidingBoxUnkF4;
sp40[0] = sp3C->oFaceAnglePitch;

1
src/game/behaviors/lll_sinking_rectangle.inc.c
View file

@ -11,7 +11,6 @@ void sinking_rectangular_plat_actions(f32 a0, s32 a1) {
break;
case 2:
break;
break; // Two breaks needed?
}
}

6
src/game/behaviors/scuttlebug.inc.c
View file

@ -60,9 +60,9 @@ void bhv_scuttlebug_loop(void) {
break;
case 1:
o->oForwardVel = 5.0f;
if (cur_obj_lateral_dist_from_obj_to_home(player) > 1000.0f)
angleToPlayer = angleToPlayer;
else {
if (cur_obj_lateral_dist_from_obj_to_home(player) > 1000.0f) {
//angleToPlayer = angleToPlayer;
} else {
if (o->oScuttlebugUnkF8 == 0) {
o->oScuttlebugUnkFC = 0;
angleToPlayer = obj_angle_to_object(o, player);

7
src/game/behaviors/spindrift.inc.c
View file

@ -29,10 +29,11 @@ void bhv_spindrift_loop(void) {
switch (o->oAction) {
case 0:
approach_forward_vel(&o->oForwardVel, 4.0f, 1.0f);
if (cur_obj_lateral_dist_from_mario_to_home() > 1000.0f)
if (cur_obj_lateral_dist_from_mario_to_home() > 1000.0f) {
angleToPlayer = cur_obj_angle_to_home();
else if (distanceToPlayer > 300.0f)
angleToPlayer = angleToPlayer;
} else if (distanceToPlayer > 300.0f) {
//angleToPlayer = angleToPlayer;
}
cur_obj_rotate_yaw_toward(angleToPlayer, 0x400);
break;
case 1:

2
src/game/behaviors/tuxie.inc.c
View file

@ -120,7 +120,7 @@ void tuxies_mother_act_0(void) {
cur_obj_init_animation_with_sound(3);
if (sp28 < 500.0f)
sp2C = 1;
if (sp24->heldByPlayerIndex == 0 && sp24 != NULL && sp28 < 300.0f && sp24->oHeldState != HELD_FREE) {
if (sp24 != NULL && sp24->heldByPlayerIndex == 0 && sp28 < 300.0f && sp24->oHeldState != HELD_FREE) {
o->oAction = 1;
sp24->oSmallPenguinUnk88 = 1;
o->prevObj = sp24;

2
src/game/interaction.c
View file

@ -1260,7 +1260,7 @@ static u8 resolve_player_collision(struct MarioState* m, struct MarioState* m2)
if (m2->playerIndex == 0) {
m2->squishTimer = max(m2->squishTimer, 4);
}
f32 velY = fmax(fmin(50.0f, 10.0f + fabs(m->vel[1])), 30.0f);
f32 velY;
if (m2->action == ACT_CROUCHING) {
mario_stop_riding_and_holding(m);
set_mario_action(m, ACT_TWIRLING, 0);

2
src/game/interaction.h
View file

@ -37,7 +37,7 @@ enum InteractionType {
INTERACT_SNUFIT_BULLET = /* 0x10000000 */ (1 << 28),
INTERACT_SHOCK = /* 0x20000000 */ (1 << 29),
INTERACT_IGLOO_BARRIER = /* 0x40000000 */ (1 << 30),
INTERACT_PLAYER = /* 0x80000000 */ (1 << 31),
INTERACT_PLAYER = /* 0x80000000 */ ((u32)1 << 31),
};
// INTERACT_WARP

2
src/game/level_info.c
View file

@ -106,7 +106,7 @@ const char* get_level_name(s16 courseNum, s16 levelNum, s16 areaIndex) {
}
// If we are in in Course 0 we are in the castle which doesn't have a string
if (courseNum && courseNum > 0 && courseNum < 27) {
if (courseNum > 0 && courseNum < 27) {
void** courseNameTbl;
#ifndef VERSION_EU

3
src/game/level_update.c
View file

@ -385,8 +385,6 @@ void set_mario_initial_action(struct MarioState *m, u32 spawnType, u32 actionArg
set_mario_initial_cap_powerup(m);
}
#include "pc/debuglog.h"
void init_mario_after_warp(void) {
struct ObjectWarpNode *spawnNode = area_get_warp_node(sWarpDest.nodeId);
u32 marioSpawnType = get_mario_spawn_type(spawnNode->object);
@ -442,7 +440,6 @@ void init_mario_after_warp(void) {
gMarioState[0].marioObj->oPosY = spawnNode->object->oPosY;
gMarioState[0].marioObj->oPosZ = spawnNode->object->oPosZ;
}
LOG_INFO(">>> override offset spawn <<<");
}
gMarioState->interactObj = spawnNode->object;

4
src/game/mario_actions_airborne.c
View file

@ -115,12 +115,14 @@ s32 check_kick_or_dive_in_air(struct MarioState *m) {
}
s32 should_get_stuck_in_ground(struct MarioState *m) {
if (m->floor == NULL) { return FALSE; }
u32 terrainType = m->area->terrainType & TERRAIN_MASK;
struct Surface *floor = m->floor;
s32 flags = floor->flags;
s32 type = floor->type;
if (floor != NULL && (terrainType == TERRAIN_SNOW || terrainType == TERRAIN_SAND)
if ((terrainType == TERRAIN_SNOW || terrainType == TERRAIN_SAND)
&& type != SURFACE_BURNING && SURFACE_IS_NOT_HARD(type)) {
if (!(flags & 0x01) && m->peakHeight - m->pos[1] > 1000.0f && floor->normal.y >= 0.8660254f) {
return TRUE;

2
src/game/mario_actions_automatic.c
View file

@ -1024,7 +1024,7 @@ s32 act_bubbled(struct MarioState* m) {
if (gLocalBubbleCounter > 0) { gLocalBubbleCounter--; }
// pop bubble
if (m->playerIndex == 0 && distanceToPlayer < 120 && is_player_active(targetMarioState) && m->numLives != -1 && gLocalBubbleCounter <= 0) {
if (m->playerIndex == 0 && distanceToPlayer < 120 && is_player_active(targetMarioState) && m->numLives != -1 && gLocalBubbleCounter == 0) {
m->marioObj->activeFlags &= ~ACTIVE_FLAG_MOVE_THROUGH_GRATE;
m->hurtCounter = 0;
m->healCounter = 31;

2
src/game/object_list_processor.c
View file

@ -533,7 +533,7 @@ void spawn_objects_from_info(UNUSED s32 unused, struct SpawnInfo *spawnInfo) {
object->respawnInfo = &spawnInfo->behaviorArg;
// found a player
if (spawnInfo->behaviorArg & (1 << 31) && object->behavior == bhvMario) {
if (spawnInfo->behaviorArg & ((u32)1 << 31) && object->behavior == bhvMario) {
u16 playerIndex = (spawnInfo->behaviorArg & ~(1 << 31));
object->oBehParams = playerIndex + 1;
gMarioObjects[playerIndex] = object;

4
src/pc/djui/djui_checkbox.c
View file

@ -33,9 +33,11 @@ static void djui_checkbox_get_cursor_hover_location(struct DjuiBase* base, f32*
static void djui_checkbox_on_cursor_down_begin(struct DjuiBase* base, UNUSED bool inputCursor) {
struct DjuiCheckbox* checkbox = (struct DjuiCheckbox*)base;
if (checkbox == NULL) { return; }
*checkbox->value = !(*checkbox->value);
djui_base_set_visible(&checkbox->rectValue->base, *checkbox->value);
if (base != NULL && base->interactable != NULL && base->interactable->on_value_change != NULL) {
if (base->interactable != NULL && base->interactable->on_value_change != NULL) {
base->interactable->on_value_change(base);
}
}

18
src/pc/lua/smlua_functions.c
View file

@ -66,17 +66,17 @@ int smlua_func_atan2s(lua_State* L) {
}
int smlua_func_init_mario_after_warp(lua_State* L) {
if (network_player_connected_count() < 2) {
if(!smlua_functions_valid_param_count(L, 0)) { return 0; }
init_mario_after_warp();
return 1;
} else {
if (network_player_connected_count() >= 2) {
LOG_LUA("This function can only be used in single-player");
return 0;
}
return 0;
if(!smlua_functions_valid_param_count(L, 0)) { return 0; }
extern void init_mario_after_warp(void);
init_mario_after_warp();
return 1;
}
void smlua_bind_functions(void) {