2021-09-20 03:13 ftp://ftp.de.debian.org/debian/pool/main/m/mirrormagic/mirrormagic_3.0.0+dfsg2.orig.tar.gz cppcheck-options: --library=posix --library=gnu --library=sdl -j1 --showtime=top5 --check-library --inconclusive --enable=style,information --template=daca2 -rp=temp -D__GNUC__ --platform=unix64 temp platform: Linux-5.4.0-84-generic-x86_64-with-glibc2.29 python: 3.8.10 client-version: 1.3.12 compiler: g++ (Ubuntu 10.3.0-1ubuntu1~20.04) 10.3.0 cppcheck: head 2.5 head-info: ee475b181 (2021-09-19 20:22:30 +0200) count: 275 TO! elapsed-time: 1028.0 1800.2 head-timing-info: Tokenizer::simplifyTokens1: 204.865s (avg. 2.59323s - 79 result(s)) Tokenizer::simplifyTokens1::ValueFlow: 193.749s (avg. 2.58332s - 75 result(s)) Tokenizer::simplifyTokens1::createSymbolDatabase: 4.90752s (avg. 0.0654336s - 75 result(s)) Other::runChecks: 2.60848s (avg. 0.0347797s - 75 result(s)) UnusedVar::runChecks: 1.38926s (avg. 0.0185235s - 75 result(s)) Overall time: 414.625s old-timing-info: head results: mirrormagic-3.0.0/src/anim.c:330:37: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:330:37: note: Shadow variable mirrormagic-3.0.0/src/anim.c:368:9: style: Local variable 'control' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/anim.c:332:7: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:368:9: note: Shadow variable mirrormagic-3.0.0/src/anim.c:429:41: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:429:41: note: Shadow variable mirrormagic-3.0.0/src/anim.c:543:41: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:543:41: note: Shadow variable mirrormagic-3.0.0/src/anim.c:697:41: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:697:41: note: Shadow variable mirrormagic-3.0.0/src/anim.c:1009:37: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:1009:37: note: Shadow variable mirrormagic-3.0.0/src/anim.c:1453:41: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:1453:41: note: Shadow variable mirrormagic-3.0.0/src/anim.c:1480:41: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/anim.c:1480:41: note: Shadow variable mirrormagic-3.0.0/src/conf_act.c:21:7: style: struct member 'Anonymous0::element' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_act.c:22:7: style: struct member 'Anonymous0::element_active' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_act.c:327:7: style: struct member 'Anonymous1::button' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_act.c:328:7: style: struct member 'Anonymous1::button_active' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_act.c:409:7: style: struct member 'Anonymous2::font_nr' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_act.c:410:7: style: struct member 'Anonymous2::font_nr_active' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_chr.c:765:40: error: syntax error [syntaxError] mirrormagic-3.0.0/src/conf_cus.c:2065:41: error: syntax error [syntaxError] mirrormagic-3.0.0/src/conf_e2g.c:21:7: style: struct member 'Anonymous0::element' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_e2g.c:22:7: style: struct member 'Anonymous0::action' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_e2g.c:23:7: style: struct member 'Anonymous0::direction' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_e2g.c:26:7: style: struct member 'Anonymous0::graphic' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_e2s.c:21:7: style: struct member 'Anonymous0::element' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_e2s.c:23:7: style: struct member 'Anonymous0::action' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_e2s.c:25:7: style: struct member 'Anonymous0::sound' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_esg.c:21:7: style: struct member 'Anonymous0::element' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_esg.c:22:7: style: struct member 'Anonymous0::special' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_esg.c:24:7: style: struct member 'Anonymous0::graphic' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_fnt.c:21:7: style: struct member 'Anonymous0::font_nr' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_fnt.c:22:7: style: struct member 'Anonymous0::special' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_fnt.c:24:7: style: struct member 'Anonymous0::graphic' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_g2m.c:21:7: style: struct member 'Anonymous0::gamemode' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_g2m.c:23:7: style: struct member 'Anonymous0::music' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_g2s.c:21:7: style: struct member 'Anonymous0::gamemode' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_g2s.c:23:7: style: struct member 'Anonymous0::sound' is never used. [unusedStructMember] mirrormagic-3.0.0/src/conf_grp.c:273:39: error: syntax error [syntaxError] mirrormagic-3.0.0/src/editor.c:10751:38: error: Array 'IntelliDrawBuffer[128][128]' accessed at index IntelliDrawBuffer[-1][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/editor.c:11530:30: note: Calling function 'SetElementIntelliDraw', 2nd argument '-1' value is -1 mirrormagic-3.0.0/src/editor.c:10751:38: note: Negative array index mirrormagic-3.0.0/src/editor.c:13019:20: warning: Either the condition 'type_id<=1' is redundant or the array 'radiobutton_info[2]' is accessed at index 2, which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/editor.c:13024:15: note: Assuming that condition 'type_id<=1' is not redundant mirrormagic-3.0.0/src/editor.c:13019:20: note: Array index out of bounds mirrormagic-3.0.0/src/editor.c:13019:20: warning: Either the condition 'type_id>=0' is redundant or the array 'radiobutton_info[2]' is accessed at index -1, which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/editor.c:13023:15: note: Assuming that condition 'type_id>=0' is not redundant mirrormagic-3.0.0/src/editor.c:13019:20: note: Negative array index mirrormagic-3.0.0/src/editor.c:13020:21: warning: Either the condition 'type_id<=1' is redundant or the array 'radiobutton_info[2]' is accessed at index 2, which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/editor.c:13024:15: note: Assuming that condition 'type_id<=1' is not redundant mirrormagic-3.0.0/src/editor.c:13020:21: note: Array index out of bounds mirrormagic-3.0.0/src/editor.c:13020:21: warning: Either the condition 'type_id>=0' is redundant or the array 'radiobutton_info[2]' is accessed at index -1, which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/editor.c:13023:15: note: Assuming that condition 'type_id>=0' is not redundant mirrormagic-3.0.0/src/editor.c:13020:21: note: Negative array index mirrormagic-3.0.0/src/editor.c:13034:20: warning: Either the condition 'type_id<=63' is redundant or the array 'checkbutton_info[64]' is accessed at index 64, which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/editor.c:13042:11: note: Assuming that condition 'type_id<=63' is not redundant mirrormagic-3.0.0/src/editor.c:13034:20: note: Array index out of bounds mirrormagic-3.0.0/src/editor.c:5402:11: warning: Either the condition 'gi==NULL' is redundant or there is possible null pointer dereference: gi. [nullPointerRedundantCheck] mirrormagic-3.0.0/src/editor.c:6360:14: note: Assuming that condition 'gi==NULL' is not redundant mirrormagic-3.0.0/src/editor.c:6365:23: note: Calling function 'getRightGadgetBorder', 1st argument 'gi' value is 0 mirrormagic-3.0.0/src/editor.c:5402:11: note: Null pointer dereference mirrormagic-3.0.0/src/editor.c:6367:12: warning: Either the condition 'gi==NULL' is redundant or there is possible null pointer dereference: gi. [nullPointerRedundantCheck] mirrormagic-3.0.0/src/editor.c:6360:14: note: Assuming that condition 'gi==NULL' is not redundant mirrormagic-3.0.0/src/editor.c:6367:12: note: Null pointer dereference mirrormagic-3.0.0/src/editor.c:6428:7: warning: Either the condition 'gi==NULL' is redundant or there is possible null pointer dereference: gi. [nullPointerRedundantCheck] mirrormagic-3.0.0/src/editor.c:6421:9: note: Assuming that condition 'gi==NULL' is not redundant mirrormagic-3.0.0/src/editor.c:6428:7: note: Null pointer dereference mirrormagic-3.0.0/src/editor.c:12041:20: style: Redundant initialization for 'letter_element'. The initialized value is overwritten before it is read. [redundantInitialization] mirrormagic-3.0.0/src/editor.c:12036:22: note: letter_element is initialized mirrormagic-3.0.0/src/editor.c:12041:20: note: letter_element is overwritten mirrormagic-3.0.0/src/editor.c:12487:13: style: Local variable 'last_sx' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/editor.c:12299:14: note: Shadowed declaration mirrormagic-3.0.0/src/editor.c:12487:13: note: Shadow variable mirrormagic-3.0.0/src/editor.c:12488:13: style: Local variable 'last_sy' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/editor.c:12299:28: note: Shadowed declaration mirrormagic-3.0.0/src/editor.c:12488:13: note: Shadow variable mirrormagic-3.0.0/src/editor.c:12527:13: style: Local variable 'last_sx' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/editor.c:12299:14: note: Shadowed declaration mirrormagic-3.0.0/src/editor.c:12527:13: note: Shadow variable mirrormagic-3.0.0/src/editor.c:12528:13: style: Local variable 'last_sy' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/editor.c:12299:28: note: Shadowed declaration mirrormagic-3.0.0/src/editor.c:12528:13: note: Shadow variable mirrormagic-3.0.0/src/editor.c:13588:6: style: Local variable 'new_element' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/editor.c:13154:7: note: Shadowed declaration mirrormagic-3.0.0/src/editor.c:13588:6: note: Shadow variable mirrormagic-3.0.0/src/editor.c:8807:9: style: Variable 'drawing_elements' can be declared with const [constVariable] mirrormagic-3.0.0/src/editor.c:2025:8: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/editor.c:7987:6: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/editor.c:7994:6: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/editor.c:8025:6: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/editor.c:8031:6: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/editor.c:2025:8: error: Signed integer overflow for expression '1<<31'. [integerOverflow] mirrormagic-3.0.0/src/editor.c:7987:6: error: Signed integer overflow for expression '1<<(63%32)'. [integerOverflow] mirrormagic-3.0.0/src/editor.c:7994:6: error: Signed integer overflow for expression '1<<(63%32)'. [integerOverflow] mirrormagic-3.0.0/src/editor.c:8025:6: error: Signed integer overflow for expression '1<<(31%32)'. [integerOverflow] mirrormagic-3.0.0/src/editor.c:8031:6: error: Signed integer overflow for expression '1<<(31%32)'. [integerOverflow] mirrormagic-3.0.0/src/events.c:1518:7: style: Condition 'handle_gadgets' is always true [knownConditionTrueFalse] mirrormagic-3.0.0/src/events.c:1494:28: note: Assignment 'handle_gadgets=1', assigned value is 1 mirrormagic-3.0.0/src/events.c:1518:7: note: Condition 'handle_gadgets' is always true mirrormagic-3.0.0/src/events.c:2263:7: style: Local variable 'joystick' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1479:28: note: Shadowed declaration mirrormagic-3.0.0/src/events.c:2263:7: note: Shadow variable mirrormagic-3.0.0/src/files.c:1403:20: portability: Non reentrant function 'localtime' called. For threadsafe applications it is recommended to use the reentrant replacement function 'localtime_r'. [prohibitedlocaltimeCalled] mirrormagic-3.0.0/src/files.c:1532:7: style:inconclusive: Found duplicate branches for 'if' and 'else'. [duplicateBranch] mirrormagic-3.0.0/src/files.c:1534:7: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:1532:7: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:1584:7: style:inconclusive: Found duplicate branches for 'if' and 'else'. [duplicateBranch] mirrormagic-3.0.0/src/files.c:1586:7: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:1584:7: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:3078:2: style:inconclusive: Found duplicate branches for 'if' and 'else'. [duplicateBranch] mirrormagic-3.0.0/src/files.c:3080:2: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:3078:2: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:9975:4: style:inconclusive: Found duplicate branches for 'if' and 'else'. [duplicateBranch] mirrormagic-3.0.0/src/files.c:9977:4: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:9975:4: note: Found duplicate branches for 'if' and 'else'. mirrormagic-3.0.0/src/files.c:6416:4: style: Same value in both branches of ternary operator. [duplicateValueTernary] mirrormagic-3.0.0/src/files.c:6427:4: style: Same value in both branches of ternary operator. [duplicateValueTernary] mirrormagic-3.0.0/src/files.c:7146:68: style: Same expression in both branches of ternary operator. [duplicateExpressionTernary] mirrormagic-3.0.0/src/files.c:9742:13: style: Local variable 'value_1' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/files.c:9732:11: note: Shadowed declaration mirrormagic-3.0.0/src/files.c:9742:13: note: Shadow variable mirrormagic-3.0.0/src/files.c:10296:9: style: Local variable 'i' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/files.c:10238:7: note: Shadowed declaration mirrormagic-3.0.0/src/files.c:10296:9: note: Shadow variable mirrormagic-3.0.0/src/files.c:8899:37: style: Parameter 'setup_value_raw' can be declared with const [constParameter] mirrormagic-3.0.0/src/files.c:7263:57: style: Parameter 'level' can be declared with const [constParameter] mirrormagic-3.0.0/src/files.c:7250:57: style: Parameter 'level' can be declared with const [constParameter] mirrormagic-3.0.0/src/files.c:7671:12: style: Variable 'joy_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/files.c:8902:53: portability: '(void*)&setup' is of type 'void *'. When using void pointers in calculations, the behaviour is undefined. [arithOperationsOnVoidPointer] mirrormagic-3.0.0/src/files.c:2735:27: error: Shifting 32-bit value by 43 bits is undefined behaviour [shiftTooManyBits] mirrormagic-3.0.0/src/files.c:2870:27: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/files.c:2870:27: error: Signed integer overflow for expression '1<0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[128][*], which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:1800:18: note: Assuming that condition 'x>0' is not redundant mirrormagic-3.0.0/src/game.c:1801:6: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:1803:6: warning: Either the condition 'y>0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[*][128], which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:1802:18: note: Assuming that condition 'y>0' is not redundant mirrormagic-3.0.0/src/game.c:1803:6: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:1805:6: warning: Either the condition 'y>0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[*][128], which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:1804:18: note: Assuming that condition 'y>0' is not redundant mirrormagic-3.0.0/src/game.c:1805:6: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:1807:6: warning: Either the condition 'y>0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[*][128], which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:1806:18: note: Assuming that condition 'y>0' is not redundant mirrormagic-3.0.0/src/game.c:1807:6: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:3109:30: warning: Either the condition '(i)<=615' is redundant or the array 'push_delay_list[7]' is accessed at index 616, which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:3137:10: note: Assuming that condition '(i)<=615' is not redundant mirrormagic-3.0.0/src/game.c:3118:27: note: Assuming condition is false mirrormagic-3.0.0/src/game.c:3109:30: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:3141:33: warning: Either the condition '(i)<=615' is redundant or the array 'move_stepsize_list[13]' is accessed at index 616, which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:3152:9: note: Assuming that condition '(i)<=615' is not redundant mirrormagic-3.0.0/src/game.c:3141:33: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:3163:33: warning: Either the condition '(i)<=615' is redundant or the array 'collect_count_list[10]' is accessed at index 616, which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:3171:10: note: Assuming that condition '(i)<=615' is not redundant mirrormagic-3.0.0/src/game.c:3163:33: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:3175:36: warning: Either the condition '(i)<=615' is redundant or the array 'access_direction_list[31]' is accessed at index 616, which is out of bounds. [arrayIndexOutOfBoundsCond] mirrormagic-3.0.0/src/game.c:3182:9: note: Assuming that condition '(i)<=615' is not redundant mirrormagic-3.0.0/src/game.c:3175:36: note: Array index out of bounds mirrormagic-3.0.0/src/game.c:6545:21: warning: Either the condition '(x)>=0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[-1][*], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:6782:5: note: Assuming that condition '(x)>=0' is not redundant mirrormagic-3.0.0/src/game.c:6545:21: note: Negative array index mirrormagic-3.0.0/src/game.c:6548:28: warning: Either the condition '(x)>=0' is redundant or the array 'MovDir[128][128]' is accessed at index MovDir[-1][*], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:6782:5: note: Assuming that condition '(x)>=0' is not redundant mirrormagic-3.0.0/src/game.c:6548:28: note: Negative array index mirrormagic-3.0.0/src/game.c:7073:6: warning: Either the condition '(newy)>=0' is redundant or the array 'StorePlayer[128][128]' is accessed at index StorePlayer[-1][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:7076:6: note: Assuming that condition '(newy)>=0' is not redundant mirrormagic-3.0.0/src/game.c:7073:6: note: Negative array index mirrormagic-3.0.0/src/game.c:9254:11: warning: Either the condition '(y)>=0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[-2][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:9235:6: note: Assuming that condition '(y)>=0' is not redundant mirrormagic-3.0.0/src/game.c:9254:11: note: Negative array index mirrormagic-3.0.0/src/game.c:9254:25: warning: Either the condition '(y)>=0' is redundant or the array 'Store[128][128]' is accessed at index Store[-2][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:9235:6: note: Assuming that condition '(y)>=0' is not redundant mirrormagic-3.0.0/src/game.c:9254:25: note: Negative array index mirrormagic-3.0.0/src/game.c:9255:12: warning: Either the condition '(y)>=0' is redundant or the array 'Store[128][128]' is accessed at index Store[-2][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:9235:6: note: Assuming that condition '(y)>=0' is not redundant mirrormagic-3.0.0/src/game.c:9255:12: note: Negative array index mirrormagic-3.0.0/src/game.c:9256:13: warning: Either the condition '(y)>=0' is redundant or the array 'GfxDir[128][128]' is accessed at index GfxDir[-2][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:9235:6: note: Assuming that condition '(y)>=0' is not redundant mirrormagic-3.0.0/src/game.c:9256:13: note: Negative array index mirrormagic-3.0.0/src/game.c:9256:28: warning: Either the condition '(y)>=0' is redundant or the array 'MovDir[128][128]' is accessed at index MovDir[-2][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:9235:6: note: Assuming that condition '(y)>=0' is not redundant mirrormagic-3.0.0/src/game.c:9256:28: note: Negative array index mirrormagic-3.0.0/src/game.c:9364:9: warning: Either the condition '(ax+1)>=0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[-2][*], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:9356:8: note: Assuming that condition '(ax+1)>=0' is not redundant mirrormagic-3.0.0/src/game.c:9364:9: note: Negative array index mirrormagic-3.0.0/src/game.c:9466:9: warning: Either the condition '(ax+1)>=0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[-2][*], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:9459:8: note: Assuming that condition '(ax+1)>=0' is not redundant mirrormagic-3.0.0/src/game.c:9466:9: note: Negative array index mirrormagic-3.0.0/src/game.c:10747:22: error: Array 'move_xy[9]' accessed at index 10, which is out of bounds. [arrayIndexOutOfBounds] mirrormagic-3.0.0/src/game.c:10748:22: error: Array 'move_xy[9]' accessed at index 10, which is out of bounds. [arrayIndexOutOfBounds] mirrormagic-3.0.0/src/game.c:14350:7: warning: Either the condition 'new_element==-1' is redundant or the array 'element_info[0]' is accessed at index -1, which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game.c:14367:19: note: Assuming that condition 'new_element==-1' is not redundant mirrormagic-3.0.0/src/game.c:14360:17: note: Assignment to 'new_element=drop_element' mirrormagic-3.0.0/src/game.c:14350:7: note: Negative array index mirrormagic-3.0.0/src/game.c:13031:9: style: Condition 'object_hit' is always true [knownConditionTrueFalse] mirrormagic-3.0.0/src/game.c:13029:18: note: Assignment 'object_hit=1', assigned value is 1 mirrormagic-3.0.0/src/game.c:13031:9: note: Condition 'object_hit' is always true mirrormagic-3.0.0/src/game.c:1789:15: style: Variable 'element' is reassigned a value before the old one has been used. [redundantAssignment] mirrormagic-3.0.0/src/game.c:1786:15: note: element is assigned mirrormagic-3.0.0/src/game.c:1789:15: note: element is overwritten mirrormagic-3.0.0/src/game.c:13029:16: style: Redundant initialization for 'object_hit'. The initialized value is overwritten before it is read. [redundantInitialization] mirrormagic-3.0.0/src/game.c:13025:24: note: object_hit is initialized mirrormagic-3.0.0/src/game.c:13029:16: note: object_hit is overwritten mirrormagic-3.0.0/src/game.c:9643:63: style: Same value in both branches of ternary operator. [duplicateValueTernary] mirrormagic-3.0.0/src/game.c:9652:44: style: Same value in both branches of ternary operator. [duplicateValueTernary] mirrormagic-3.0.0/src/game.c:9661:41: style: Same value in both branches of ternary operator. [duplicateValueTernary] mirrormagic-3.0.0/src/game.c:12296:3: style:inconclusive: Statements following return, break, continue, goto or throw will never be executed. [unreachableCode] mirrormagic-3.0.0/src/game.c:5932:9: style: Local variable 'element' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game.c:5914:7: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:5932:9: note: Shadow variable mirrormagic-3.0.0/src/game.c:5950:9: style: Local variable 'element' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game.c:5914:7: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:5950:9: note: Shadow variable mirrormagic-3.0.0/src/game.c:9733:11: style: Local variable 'element' shadows outer argument [shadowArgument] mirrormagic-3.0.0/src/game.c:9602:58: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:9733:11: note: Shadow variable mirrormagic-3.0.0/src/game.c:9833:11: style: Local variable 'element' shadows outer argument [shadowArgument] mirrormagic-3.0.0/src/game.c:9602:58: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:9833:11: note: Shadow variable mirrormagic-3.0.0/src/game.c:9982:10: style: Local variable 'element' shadows outer argument [shadowArgument] mirrormagic-3.0.0/src/game.c:9602:58: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:9982:10: note: Shadow variable mirrormagic-3.0.0/src/game.c:10043:7: style: Local variable 'element' shadows outer argument [shadowArgument] mirrormagic-3.0.0/src/game.c:9602:58: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:10043:7: note: Shadow variable mirrormagic-3.0.0/src/game.c:11736:11: style: Local variable 'x' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game.c:11642:10: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:11736:11: note: Shadow variable mirrormagic-3.0.0/src/game.c:11737:11: style: Local variable 'y' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game.c:11642:13: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:11737:11: note: Shadow variable mirrormagic-3.0.0/src/game.c:12027:11: style: Local variable 'element' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game.c:11642:16: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:12027:11: note: Shadow variable mirrormagic-3.0.0/src/game.c:15433:25: style: Local variable 'gfx' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1474:24: note: Shadowed declaration mirrormagic-3.0.0/src/game.c:15433:25: note: Shadow variable mirrormagic-3.0.0/src/game.c:4945:39: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game.c:4946:39: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game.c:12244:32: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game.c:12245:32: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game.c:12260:32: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game.c:12261:32: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game.c:12161:53: style: Parameter 'player' can be declared with const [constParameter] mirrormagic-3.0.0/src/game.c:11060:13: style: Parameter 'tape_action' can be declared with const [constParameter] mirrormagic-3.0.0/src/game.c:15029:14: style: Variable 'belt_base_active_element' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:13145:14: style: Variable 'test_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:13138:14: style: Variable 'touch_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:13073:14: style: Variable 'test_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:12924:14: style: Variable 'touch_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:12824:14: style: Variable 'touch_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:7261:16: style: Variable 'test_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:6949:18: style: Variable 'check_pos' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:5906:14: style: Variable 'belt_move_dir' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:5899:14: style: Variable 'belt_base_switch_element' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:5892:14: style: Variable 'belt_base_active_element' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:5885:14: style: Variable 'belt_base_element' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:5827:14: style: Variable 'belt_base_active_element' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:5820:14: style: Variable 'belt_base_element' can be declared with const [constVariable] mirrormagic-3.0.0/src/game.c:8471:7: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/game.c:8471:7: error: Signed integer overflow for expression '1<<(31%32)'. [integerOverflow] mirrormagic-3.0.0/src/game.c:15334:7: style: struct member 'Anonymous13::gadget_id' is never used. [unusedStructMember] mirrormagic-3.0.0/src/game.c:7732:19: style: Variable 'can_fall_right' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/game.c:7736:16: style: Variable 'can_fall_both' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/game_em/convert.c:455:3: style: Assignment of function parameter has no effect outside the function. [uselessAssignmentArg] mirrormagic-3.0.0/src/game_em/convert.c:404:5: style: Opposite expression on both sides of '&='. [oppositeExpression] mirrormagic-3.0.0/src/game_em/convert.c:1084:6: style: Local variable 'x' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_em/convert.c:1000:10: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/convert.c:1084:6: note: Shadow variable mirrormagic-3.0.0/src/game_em/convert.c:1085:6: style: Local variable 'y' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_em/convert.c:1000:13: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/convert.c:1085:6: note: Shadow variable mirrormagic-3.0.0/src/game_em/convert.c:647:14: style: Variable 'eater_offset' can be declared with const [constVariable] mirrormagic-3.0.0/src/game_em/convert.c:455:10: style: Variable 'length' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/game_em/synchro_1.c:470:65: style: Same value in both branches of ternary operator. [duplicateValueTernary] mirrormagic-3.0.0/src/game_em/synchro_1.c:371:9: style: Local variable 'oldx' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_em/synchro_1.c:339:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/synchro_1.c:371:9: note: Shadow variable mirrormagic-3.0.0/src/game_em/synchro_1.c:372:9: style: Local variable 'oldy' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_em/synchro_1.c:340:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/synchro_1.c:372:9: note: Shadow variable mirrormagic-3.0.0/src/game_em/synchro_1.c:373:9: style: Local variable 'x' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_em/synchro_1.c:341:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/synchro_1.c:373:9: note: Shadow variable mirrormagic-3.0.0/src/game_em/synchro_1.c:374:9: style: Local variable 'y' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_em/synchro_1.c:342:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/synchro_1.c:374:9: note: Shadow variable mirrormagic-3.0.0/src/game_em/synchro_1.c:470:7: style: Local variable 'anim' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1477:25: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/synchro_1.c:470:7: note: Shadow variable mirrormagic-3.0.0/src/game_em/synchro_2.c:4874:11: style: Local variable 'temp' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_em/synchro_2.c:50:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_em/synchro_2.c:4874:11: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3511:5: style:inconclusive: Consecutive return, break, continue, goto or throw statements are unnecessary. [duplicateBreak] mirrormagic-3.0.0/src/game_mm/mm_game.c:1136:11: style: Local variable 'element' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:1074:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:1136:11: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3161:11: style: Local variable 'i' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:13: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3161:11: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3301:11: style: Local variable 'i' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:13: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3301:11: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3584:11: style: Local variable 'x' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3584:11: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3584:14: style: Local variable 'y' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:10: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3584:14: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3646:11: style: Local variable 'x' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3646:11: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3646:20: style: Local variable 'y' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:10: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3646:20: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3684:9: style: Local variable 'x' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:7: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3684:9: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:3684:12: style: Local variable 'y' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3094:10: note: Shadowed declaration mirrormagic-3.0.0/src/game_mm/mm_game.c:3684:12: note: Shadow variable mirrormagic-3.0.0/src/game_mm/mm_game.c:2606:9: style: struct member 'Anonymous4::left' is never used. [unusedStructMember] mirrormagic-3.0.0/src/game_mm/mm_game.c:3641:8: style: Variable 'de' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3642:8: style: Variable 'dm' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/game_mm/mm_game.c:3981:15: style: Variable 'hi_pos' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/game_mm/mm_init.c:174:15: style: Variable 'ep_bit' can be declared with const [constVariable] mirrormagic-3.0.0/src/game_mm/mm_tools.c:338:21: warning: Either the condition '(y)>=0' is redundant or the array 'Feld[128][128]' is accessed at index Feld[-1][-1], which is out of bounds. [negativeIndex] mirrormagic-3.0.0/src/game_mm/mm_tools.c:340:8: note: Assuming that condition '(y)>=0' is not redundant mirrormagic-3.0.0/src/game_mm/mm_tools.c:338:21: note: Negative array index mirrormagic-3.0.0/src/game_sp/DDScrollBuffer.c:332:11: style: Variable 'ScrollX' is reassigned a value before the old one has been used. [redundantAssignment] mirrormagic-3.0.0/src/game_sp/DDScrollBuffer.c:328:13: note: ScrollX is assigned mirrormagic-3.0.0/src/game_sp/DDScrollBuffer.c:332:11: note: ScrollX is overwritten mirrormagic-3.0.0/src/game_sp/DDScrollBuffer.c:333:11: style: Variable 'ScrollY' is reassigned a value before the old one has been used. [redundantAssignment] mirrormagic-3.0.0/src/game_sp/DDScrollBuffer.c:329:13: note: ScrollY is assigned mirrormagic-3.0.0/src/game_sp/DDScrollBuffer.c:333:11: note: ScrollY is overwritten mirrormagic-3.0.0/src/game_sp/Input.c:12:26: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Input.c:13:20: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Input.c:14:20: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Input.c:15:21: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Input.c:19:26: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Input.c:20:20: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Input.c:21:20: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Input.c:22:21: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/game_sp/Murphy.c:452:6: style: Variable 'al' is reassigned a value before the old one has been used. [redundantAssignment] mirrormagic-3.0.0/src/game_sp/Murphy.c:451:6: note: al is assigned mirrormagic-3.0.0/src/game_sp/Murphy.c:452:6: note: al is overwritten mirrormagic-3.0.0/src/game_sp/file.c:244:9: style: Local variable 'level_nr' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1485:14: note: Shadowed declaration mirrormagic-3.0.0/src/game_sp/file.c:244:9: note: Shadow variable mirrormagic-3.0.0/src/game_sp/main.c:75:26: style: Parameter 'action' can be declared with const [constParameter] mirrormagic-3.0.0/src/init.c:126:29: warning: The address of local variable 'anim_initial' might be accessed at non-zero index. [objectIndex] mirrormagic-3.0.0/src/init.c:120:18: note: Address of variable taken here. mirrormagic-3.0.0/src/init.c:126:29: note: The address of local variable 'anim_initial' might be accessed at non-zero index. mirrormagic-3.0.0/src/init.c:127:30: warning: The address of local variable 'anim_initial' might be accessed at non-zero index. [objectIndex] mirrormagic-3.0.0/src/init.c:120:18: note: Address of variable taken here. mirrormagic-3.0.0/src/init.c:127:30: note: The address of local variable 'anim_initial' might be accessed at non-zero index. mirrormagic-3.0.0/src/init.c:1377:7: style: The if condition is the same as the previous if condition [duplicateCondition] mirrormagic-3.0.0/src/init.c:1354:7: note: First condition mirrormagic-3.0.0/src/init.c:1377:7: note: Second condition mirrormagic-3.0.0/src/init.c:448:6: style: Local variable 'j' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/init.c:342:10: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:448:6: note: Shadow variable mirrormagic-3.0.0/src/init.c:497:6: style: Local variable 'base_graphic' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/init.c:480:11: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:497:6: note: Shadow variable mirrormagic-3.0.0/src/init.c:498:6: style: Local variable 'active_graphic' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/init.c:481:11: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:498:6: note: Shadow variable mirrormagic-3.0.0/src/init.c:499:10: style: Local variable 'base_redefined' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/init.c:482:15: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:499:10: note: Shadow variable mirrormagic-3.0.0/src/init.c:501:10: style: Local variable 'active_redefined' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/init.c:484:15: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:501:10: note: Shadow variable mirrormagic-3.0.0/src/init.c:593:9: style: Local variable 'j' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/init.c:579:10: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:593:9: note: Shadow variable mirrormagic-3.0.0/src/init.c:594:9: style: Local variable 'k' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/init.c:579:13: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:594:9: note: Shadow variable mirrormagic-3.0.0/src/init.c:2108:9: style: Local variable 'level' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/main.h:3611:26: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:2108:9: note: Shadow variable mirrormagic-3.0.0/src/init.c:4998:6: style: Local variable 'level_nr' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1485:14: note: Shadowed declaration mirrormagic-3.0.0/src/init.c:4998:6: note: Shadow variable mirrormagic-3.0.0/src/init.c:2380:34: style: Parameter 'bitfield' can be declared with const [constParameter] mirrormagic-3.0.0/src/init.c:5123:16: style: Variable 'ignore_generic_tokens' can be declared with const [constVariable] mirrormagic-3.0.0/src/init.c:4668:16: style: Variable 'ep_em_explodes_by_fire' can be declared with const [constVariable] mirrormagic-3.0.0/src/init.c:4653:16: style: Variable 'ep_em_slippery_wall' can be declared with const [constVariable] mirrormagic-3.0.0/src/init.c:4433:14: style: Variable 'no_wall_properties' can be declared with const [constVariable] mirrormagic-3.0.0/src/init.c:1643:14: style: Variable 'full_size_graphics' can be declared with const [constVariable] mirrormagic-3.0.0/src/init.c:4502:5: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/init.c:4502:5: error: Signed integer overflow for expression '1<<(63%32)'. [integerOverflow] mirrormagic-3.0.0/src/init.c:5934:21: style: Variable 'num_lines_printed' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/gadgets.c:735:7: style: Condition 'direct' is always false [knownConditionTrueFalse] mirrormagic-3.0.0/src/libgame/gadgets.c:733:12: note: Assignment 'direct=0', assigned value is 0 mirrormagic-3.0.0/src/libgame/gadgets.c:735:7: note: Condition 'direct' is always false mirrormagic-3.0.0/src/libgame/gadgets.c:1579:24: style: Local variable 'gi' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/gadgets.c:1508:31: note: Shadowed declaration mirrormagic-3.0.0/src/libgame/gadgets.c:1579:24: note: Shadow variable mirrormagic-3.0.0/src/libgame/gadgets.c:1684:9: style: Local variable 'last_mx' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/gadgets.c:1503:14: note: Shadowed declaration mirrormagic-3.0.0/src/libgame/gadgets.c:1684:9: note: Shadow variable mirrormagic-3.0.0/src/libgame/gadgets.c:1685:9: style: Local variable 'last_my' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/gadgets.c:1503:27: note: Shadowed declaration mirrormagic-3.0.0/src/libgame/gadgets.c:1685:9: note: Shadow variable mirrormagic-3.0.0/src/libgame/misc.c:579:16: portability: Non reentrant function 'getpwuid' called. For threadsafe applications it is recommended to use the reentrant replacement function 'getpwuid_r'. [prohibitedgetpwuidCalled] mirrormagic-3.0.0/src/libgame/misc.c:611:16: portability: Non reentrant function 'getpwuid' called. For threadsafe applications it is recommended to use the reentrant replacement function 'getpwuid_r'. [prohibitedgetpwuidCalled] mirrormagic-3.0.0/src/libgame/misc.c:2433:30: portability: Non reentrant function 'readdir' called. For threadsafe applications it is recommended to use the reentrant replacement function 'readdir_r'. [prohibitedreaddirCalled] mirrormagic-3.0.0/src/libgame/misc.c:1810:22: style: Condition 'c1<='9'' is always true [knownConditionTrueFalse] mirrormagic-3.0.0/src/libgame/misc.c:1809:28: note: Assuming that condition 'c1<='9'' is not redundant mirrormagic-3.0.0/src/libgame/misc.c:1810:22: note: Condition 'c1<='9'' is always true mirrormagic-3.0.0/src/libgame/misc.c:304:3: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] mirrormagic-3.0.0/src/libgame/misc.c:267:15: style: Variable 'shift_array' can be declared with const [constVariable] mirrormagic-3.0.0/src/libgame/sdl.c:2040:6: style: Variable 'sp' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/sdl.c:2112:6: style: Variable 'sp' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/sdl.c:2192:7: style: Variable 'csx' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/sdl.c:2196:9: style: Variable 'csx' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/sdl.c:2200:7: style: Variable 'csy' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/sdl.c:2204:9: style: Variable 'csy' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/sdl.c:2209:6: style: Variable 'sp' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/setup.c:1415:18: portability: Non reentrant function 'getpwuid' called. For threadsafe applications it is recommended to use the reentrant replacement function 'getpwuid_r'. [prohibitedgetpwuidCalled] mirrormagic-3.0.0/src/libgame/setup.c:2832:7: style: Condition '!use_artworkinfo_cache' is always false [knownConditionTrueFalse] mirrormagic-3.0.0/src/libgame/setup.c:2901:7: style: Condition 'set_cache_timestamps' is always true [knownConditionTrueFalse] mirrormagic-3.0.0/src/libgame/setup.c:2896:34: note: Assignment 'set_cache_timestamps=1', assigned value is 1 mirrormagic-3.0.0/src/libgame/setup.c:2901:7: note: Condition 'set_cache_timestamps' is always true mirrormagic-3.0.0/src/libgame/setup.c:4116:17: warning: Either the condition 'value!=NULL' is redundant or there is possible null pointer dereference: value. [nullPointerRedundantCheck] mirrormagic-3.0.0/src/libgame/setup.c:4113:12: note: Assuming that condition 'value!=NULL' is not redundant mirrormagic-3.0.0/src/libgame/setup.c:4116:17: note: Null pointer dereference mirrormagic-3.0.0/src/libgame/setup.c:4058:16: style: Redundant initialization for 'level_subdir'. The initialized value is overwritten before it is read. [redundantInitialization] mirrormagic-3.0.0/src/libgame/setup.c:4040:22: note: level_subdir is initialized mirrormagic-3.0.0/src/libgame/setup.c:4058:16: note: level_subdir is overwritten mirrormagic-3.0.0/src/libgame/setup.c:4086:11: style: Local variable 'level_nr' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1485:14: note: Shadowed declaration mirrormagic-3.0.0/src/libgame/setup.c:4086:11: note: Shadow variable mirrormagic-3.0.0/src/libgame/setup.c:4111:6: style: Local variable 'level_nr' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1485:14: note: Shadowed declaration mirrormagic-3.0.0/src/libgame/setup.c:4111:6: note: Shadow variable mirrormagic-3.0.0/src/libgame/setup.c:1888:14: style: Variable 'line_raw' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/snapshot.c:98:49: style: Parameter 'node' can be declared with const [constParameter] mirrormagic-3.0.0/src/libgame/sound.c:70:11: style: struct member 'AudioFormatInfo::stereo' is never used. [unusedStructMember] mirrormagic-3.0.0/src/libgame/sound.c:72:7: style: struct member 'AudioFormatInfo::sample_rate' is never used. [unusedStructMember] mirrormagic-3.0.0/src/libgame/sound.c:73:7: style: struct member 'AudioFormatInfo::fragment_size' is never used. [unusedStructMember] mirrormagic-3.0.0/src/libgame/system.c:1482:31: style: Condition 'bit_order_msb' is always true [knownConditionTrueFalse] mirrormagic-3.0.0/src/libgame/system.c:1468:45: note: Assignment 'bit_order_msb=cursor_bit_order==0', assigned value is 1 mirrormagic-3.0.0/src/libgame/system.c:1482:31: note: Condition 'bit_order_msb' is always true mirrormagic-3.0.0/src/libgame/system.c:554:51: style: Parameter 'src_bitmap' can be declared with const [constParameter] mirrormagic-3.0.0/src/libgame/system.c:1235:12: style: Variable 'height_0' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/libgame/text.c:534:37: style: Condition 'current_ypos=pos_start' is always true [knownConditionTrueFalse] mirrormagic-3.0.0/src/screens.c:6385:12: note: Assuming that condition '(y)>=0' is not redundant mirrormagic-3.0.0/src/screens.c:6386:7: note: Condition 'y>=pos_start' is always true mirrormagic-3.0.0/src/screens.c:5910:10: style: Redundant initialization for 'startx'. The initialized value is overwritten before it is read. [redundantInitialization] mirrormagic-3.0.0/src/screens.c:5878:14: note: startx is initialized mirrormagic-3.0.0/src/screens.c:5910:10: note: startx is overwritten mirrormagic-3.0.0/src/screens.c:5911:10: style: Redundant initialization for 'starty'. The initialized value is overwritten before it is read. [redundantInitialization] mirrormagic-3.0.0/src/screens.c:5879:14: note: starty is initialized mirrormagic-3.0.0/src/screens.c:5911:10: note: starty is overwritten mirrormagic-3.0.0/src/screens.c:4021:9: style: Local variable 'num_entries' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/screens.c:4010:7: note: Shadowed declaration mirrormagic-3.0.0/src/screens.c:4021:9: note: Shadow variable mirrormagic-3.0.0/src/screens.c:6953:17: style: Local variable 'joystick' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1479:28: note: Shadowed declaration mirrormagic-3.0.0/src/screens.c:6953:17: note: Shadow variable mirrormagic-3.0.0/src/screens.c:5844:31: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/screens.c:2282:14: style: Variable 'first_entry_stores' can be declared with const [constVariable] mirrormagic-3.0.0/src/screens.c:2281:14: style: Variable 'choice_stores' can be declared with const [constVariable] mirrormagic-3.0.0/src/screens.c:2280:14: style: Variable 'num_page_entries_all_last' can be declared with const [constVariable] mirrormagic-3.0.0/src/screens.c:6251:16: style: Variable 'joystick_name' can be declared with const [constVariable] mirrormagic-3.0.0/src/screens.c:2305:9: style: Variable 'y' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/screens.c:2305:17: style: Variable 'y_old' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/screens.c:2516:9: style: Variable 'choice' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/screens.c:3060:14: style: Variable 'ystart' is assigned a value that is never used. [unreadVariable] mirrormagic-3.0.0/src/tape.c:508:19: portability: Non reentrant function 'localtime' called. For threadsafe applications it is recommended to use the reentrant replacement function 'localtime_r'. [prohibitedlocaltimeCalled] mirrormagic-3.0.0/src/tape.c:178:24: style: Local variable 'gfx' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1474:24: note: Shadowed declaration mirrormagic-3.0.0/src/tape.c:178:24: note: Shadow variable mirrormagic-3.0.0/src/tape.c:1285:25: style: Local variable 'gfx' shadows outer variable [shadowVariable] mirrormagic-3.0.0/src/libgame/system.h:1474:24: note: Shadowed declaration mirrormagic-3.0.0/src/tape.c:1285:25: note: Shadow variable mirrormagic-3.0.0/src/tape.c:301:53: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/tape.c:302:32: style: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] mirrormagic-3.0.0/src/tape.c:657:28: style: Parameter 'action' can be declared with const [constParameter] mirrormagic-3.0.0/src/tape.c:692:28: style: Parameter 'action_raw' can be declared with const [constParameter] mirrormagic-3.0.0/src/tape.c:410:20: portability: Shifting signed 32-bit value by 31 bits is implementation-defined behaviour [shiftTooManyBitsSigned] mirrormagic-3.0.0/src/tape.c:410:20: error: Signed integer overflow for expression '1<<31'. [integerOverflow] mirrormagic-3.0.0/src/tape.c:1248:7: style: struct member 'Anonymous7::gadget_id' is never used. [unusedStructMember] mirrormagic-3.0.0/src/tools.c:8363:25: error: Syntax Error: AST broken, ternary operator lacks ':'. [internalAstError] mirrormagic-3.0.0/src/editor.c:5402:11: warning: Null pointer dereference: gi [ctunullpointer] mirrormagic-3.0.0/src/editor.c:6360:14: note: Assuming that condition 'gi==NULL' is not redundant mirrormagic-3.0.0/src/editor.c:6365:22: note: Calling function getRightGadgetBorder, 1st argument is null mirrormagic-3.0.0/src/editor.c:5402:11: note: Dereferencing argument gi that is null DONE