Go through warnings and clean them up

[InFerYes]

[src\g_arty.c:288]: (style) Redundant condition: If 'EXPR == 4', the comparison 'EXPR' is always true.
[src\g_arty.c:360] -> [src\g_arty.c:367]: (style) Variable 'check->obj_owner' is reassigned a value before the old one has been used.
[src\g_arty.c:609] -> [src\g_arty.c:617]: (style) Variable 'ent->nextthink' is reassigned a value before the old one has been used.
[src\g_arty.c:1070] -> [src\g_arty.c:1076]: (style) Variable 'ent->nextthink' is reassigned a value before the old one has been used.
[src\g_combat.c:306]: (style) Assignment of function parameter has no effect outside the function.
[src\g_combat.c:793]: (style) Redundant condition: If 'EXPR == 58', the comparison 'EXPR' is always true.
[src\g_combat.c:1650] -> [src\g_combat.c:1652]: (style) Variable 'speed' is reassigned a value before the old one has been used.
[src\g_cmds.c:343]: (style) Redundant condition: If '!EXPR', the comparison 'EXPR != 120' is always true.
[src\g_cmds.c:1354]: (style) Redundant condition: it. '!it || (it && !it->use)' is equivalent to '!it || !it->use'
[src\g_cmds.c:3015]: (style) Redundant condition: If 'EXPR == 5', the comparison 'EXPR' is always true.
[src\g_cmds.c:1354] -> [src\g_cmds.c:1356]: (warning) Either the condition '!it' is redundant or there is possible null pointer dereference: it.
[src\g_cmds.c:3259]: (style) Same expression on both sides of '||'.
[src\g_cmds.c:3056]: (error) Uninitialized variable: ammo_amount
[src\g_ents.c:426]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_ents.c:441]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_ents.c:456]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_ents.c:472]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_ents.c:524]: (warning) Found calculation inside sizeof().
[src\g_items.c:794]: (style) Redundant condition: If 'EXPR == 1', the comparison 'EXPR' is always true.
[src\g_func.c:423] -> [src\g_func.c:421]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[src\g_func.c:1146] -> [src\g_func.c:1144]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[src\g_func.c:1547] -> [src\g_func.c:1545]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[src\g_func.c:2067] -> [src\g_func.c:2065]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[src\g_func.c:476] -> [src\g_func.c:482]: (style) Variable 'tmin[2]' is reassigned a value before the old one has been used.
[src\g_maps.c:118]: (warning) fscanf() without field width limits can crash with huge input data.
[src\g_maps.c:135]: (warning) fscanf() without field width limits can crash with huge input data.
[src\g_main.c:398]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_main.c:435]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_main.c:666]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_main.c:702]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_main.c:804]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_main.c:712] -> [src\g_main.c:711] -> [src\g_main.c:735] -> [src\g_main.c:737] -> [src\g_main.c:340]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[src\g_main.c:712] -> [src\g_main.c:711] -> [src\g_main.c:735] -> [src\g_main.c:737] -> [src\g_main.c:350]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[src\g_main.c:712] -> [src\g_main.c:711] -> [src\g_main.c:735] -> [src\g_main.c:737] -> [src\g_main.c:359]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[src\g_main.c:712] -> [src\g_main.c:735] -> [src\g_main.c:736]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: check.
[src\g_main.c:736]: (error) Null pointer dereference
[src\g_local.h:1033] -> [src\g_main.c:942]: (style) Local variable 'range' shadows outer function
[src\g_main.c:804]: (style) Variable 'sb' is allocated memory that is never used.
[src\g_misc.c:256] -> [src\g_misc.c:264]: (style) Variable 'gib->solid' is reassigned a value before the old one has been used.
[src\g_misc.c:1798]: (warning) Redundant assignment of 'ent->s.frame' to itself.
[src\g_misc.c:1822]: (warning) Redundant assignment of 'ent->s.frame' to itself.
[src\g_misc.c:1885]: (warning) Redundant assignment of 'ent->s.frame' to itself.
[src\g_save.c:868] -> [src\g_save.c:876]: (error) Resource handle 'f' freed twice.
[src\g_spawn.c:482]: (style) Array index 'i' is used before limits check.
[src\g_spawn.c:926]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_spawn.c:531]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[src\g_spawn.c:532]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[src\g_spawn.c:533]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[src\g_spawn.c:539]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[src\g_spawn.c:543]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[src\g_spawn.c:544]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[src\g_spawn.c:545]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[src\g_svcmds.c:444]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\g_svcmds.c:78] -> [src\g_svcmds.c:80]: (style) Variable 'ent->client->resp.mos' is reassigned a value before the old one has been used.
[src\g_turret.c:922]: (style) Redundant condition: If 'EXPR == 1', the comparison 'EXPR' is always true.
[src\g_turret.c:187]: (warning) Comparison of modulo result is predetermined, because it is always less than 2.
[src\g_turret.c:106] -> [src\g_turret.c:103]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[src\g_turret.c:936] -> [src\g_turret.c:956]: (style) Variable 'self->movetype' is reassigned a value before the old one has been used.
[src\g_turret.c:282]: (warning) Redundant assignment of 'shell->nextthink' to itself.
[src\g_turretx.c:859]: (style) Redundant condition: If 'EXPR == 1', the comparison 'EXPR' is always true.
[src\g_turretx.c:106] -> [src\g_turretx.c:103]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[src\g_turretx.c:263]: (warning) Redundant assignment of 'shell->nextthink' to itself.
[src\m_civilian.c:166] -> [src\m_civilian.c:183]: (style) Variable 'tr' is reassigned a value before the old one has been used.
[src\m_civilian.c:334] -> [src\m_civilian.c:381]: (style) Local variable 'e' shadows outer variable
[src\m_civilian.c:334] -> [src\m_civilian.c:381]: (style) Local variable 'nearest' shadows outer variable
[src\m_civilian.c:335] -> [src\m_civilian.c:382]: (style) Local variable 'temp_distance' shadows outer variable
[src\m_civilian.c:335] -> [src\m_civilian.c:382]: (style) Local variable 'nearest_distance' shadows outer variable
[src\g_weapon.c:586]: (style) Boolean result is used in bitwise operation. Clarify expression with parentheses.
[src\g_weapon.c:2035]: (style) Redundant condition: ent->owner. '!ent->owner || (ent->owner && !ent->owner->inuse)' is equivalent to '!ent->owner || !ent->owner->inuse'
[src\g_weapon.c:4310]: (style) Redundant condition: ent->owner->client->resp.team_on. '!A || (A && B)' is equivalent to '!A || B'
[src\g_weapon.c:1038] -> [src\g_weapon.c:1039]: (style) Variable 'underwater_bullet->think' is reassigned a value before the old one has been used.
[src\g_weapon.c:2157] -> [src\g_weapon.c:2165]: (style) Variable 'rocket->think' is reassigned a value before the old one has been used.
[src\g_weapon.c:2217] -> [src\g_weapon.c:2225]: (style) Variable 'rocket->think' is reassigned a value before the old one has been used.
[src\g_weapon.c:5021] -> [src\g_weapon.c:5027]: (style) Variable 'ent->client->kick_angles[0]' is reassigned a value before the old one has been used.
[src\g_weapon.c:5208] -> [src\g_weapon.c:5214]: (style) Variable 'ent->client->kick_angles[0]' is reassigned a value before the old one has been used.
[src\g_weapon.c:6170] -> [src\g_weapon.c:6176]: (style) Variable 'ent->client->kick_angles[0]' is reassigned a value before the old one has been used.
[src\g_weapon.c:2081]: (warning) Redundant assignment of 'rocket->nextthink' to itself.
[src\g_weapon.c:2171]: (warning) Redundant assignment of 'rocket->nextthink' to itself.
[src\g_weapon.c:2231]: (warning) Redundant assignment of 'rocket->nextthink' to itself.
[src\g_local.h:1033] -> [src\g_weapon.c:102]: (style) Local variable 'range' shadows outer function
[src\g_weapon.c:1349] -> [src\g_weapon.c:1410]: (style) Local variable 'i' shadows outer variable
[src\g_weapon.c:1349] -> [src\g_weapon.c:1465]: (style) Local variable 'i' shadows outer variable
[src\p_classes.c:171] -> [src\p_classes.c:168] -> [src\p_classes.c:42]: (warning) Either the condition '!item' is redundant or there is possible null pointer dereference: item.
[src\p_client.c:2964]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\p_client.c:3388]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\p_client.c:4648]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\p_client.c:4891]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[src\p_client.c:1990] -> [src\p_client.c:1999]: (style) Condition '!spot' is always true
[src\p_client.c:2039] -> [src\p_client.c:2059]: (style) Condition '!spot' is always true
[src\p_client.c:4850]: (error) Memory leak: f
[src\p_client.c:742] -> [src\p_client.c:733]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[src\p_client.c:742] -> [src\p_client.c:734]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[src\p_client.c:742] -> [src\p_client.c:736]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[src\p_client.c:742] -> [src\p_client.c:737]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[src\p_client.c:2822] -> [src\p_client.c:2815]: (warning) Either the condition 'if(ent)' is redundant or there is possible null pointer dereference: ent.
[src\p_client.c:2822] -> [src\p_client.c:2921]: (warning) Either the condition 'ent' is redundant or there is possible null pointer dereference: ent.
[src\p_client.c:2163] -> [src\p_client.c:2167]: (style) Variable 'body->svflags' is reassigned a value before the old one has been used.
[src\p_client.c:4248] -> [src\p_client.c:4250]: (style) Variable 'temp_damage' is reassigned a value before the old one has been used.
[src\p_client.c:1226] -> [src\p_client.c:1343]: (style) Local variable 'i' shadows outer variable
[src\g_local.h:1033] -> [src\p_client.c:1625]: (style) Local variable 'range' shadows outer function
[src\p_hud.c:793]: (error) Buffer is accessed out of bounds: pingstring
[src\p_hud.c:795]: (error) Buffer is accessed out of bounds: pingstring
[src\p_hud.c:801]: (error) Buffer is accessed out of bounds: pingstring
[src\p_hud.c:841]: (error) Buffer is accessed out of bounds: pingstring
[src\p_hud.c:843]: (error) Buffer is accessed out of bounds: pingstring
[src\p_hud.c:849]: (error) Buffer is accessed out of bounds: pingstring
[src\p_hud.c:180]: (style) Boolean result is used in bitwise operation. Clarify expression with parentheses.
[src\p_hud.c:1419]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_hud.c:1422]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_hud.c:1426]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_hud.c:1429]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_hud.c:1431]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_hud.c:1440]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_hud.c:1444]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_hud.c:1471]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[src\p_observer.c:267] -> [src\p_observer.c:277]: (style) Variable 'tr' is reassigned a value before the old one has been used.
[src\p_observer.c:876] -> [src\p_observer.c:887]: (style) Variable 'i' is reassigned a value before the old one has been used.
[src\p_observer.c:987] -> [src\p_observer.c:989]: (style) Variable 'ent->client->resp.mos' is reassigned a value before the old one has been used.
[src\p_weapon.c:2390]: (error) Array 'fire_frames[2]' accessed at index 2, which is out of bounds.
[src\p_weapon.c:1361]: (style) Boolean result is used in bitwise operation. Clarify expression with parentheses.
[src\p_weapon.c:1216] -> [src\p_weapon.c:1222]: (style) The if condition is the same as the previous if condition
[src\p_weapon.c:1502] -> [src\p_weapon.c:1556]: (warning) Either the condition 'surf' is redundant or there is possible null pointer dereference: surf.
[src\p_weapon.c:2053] -> [src\p_weapon.c:2048]: (warning) Either the condition 'if(ent)' is redundant or there is possible null pointer dereference: ent.
[src\p_weapon.c:922] -> [src\p_weapon.c:925]: (style) Variable 'speed' is reassigned a value before the old one has been used.
[src\p_weapon.c:2684] -> [src\p_weapon.c:2686]: (style) Variable 'ent->client->ps.gunframe' is reassigned a value before the old one has been used.
[src\p_view.c:854]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[src\p_view.c:880] -> [src\p_view.c:884]: (style) The if condition is the same as the previous if condition
[src\p_view.c:2227] -> [src\p_view.c:2231]: (style) Variable 'client->anim_end' is reassigned a value before the old one has been used.
[src\p_view.c:2425] -> [src\p_view.c:2432]: (style) Variable 'ent->viewheight' is reassigned a value before the old one has been used.
[src\p_view.c:2869] -> [src\p_view.c:2870]: (style) Variable 'dist' is reassigned a value before the old one has been used.
[src\p_view.c:50] -> [src\p_view.c:1235]: (style) Local variable 'forward' shadows outer variable
[src\p_view.c:1625] -> [src\p_view.c:1638]: (style) Local variable 'end' shadows outer variable
[src\p_view.c:1626] -> [src\p_view.c:1639]: (style) Local variable 'tr' shadows outer variable
[src\p_view.c:50] -> [src\p_view.c:2354]: (style) Local variable 'up' shadows outer variable
[src\p_view.c:50] -> [src\p_view.c:2357]: (style) Local variable 'right' shadows outer variable
[src\p_view.c:50] -> [src\p_view.c:2688]: (style) Local variable 'forward' shadows outer variable
[src\p_view.c:2355] -> [src\p_view.c:2828]: (style) Local variable 'tr' shadows outer variable
[src\p_view.c:2353] -> [src\p_view.c:2829]: (style) Local variable 'end' shadows outer variable
[src\p_view.c:2352] -> [src\p_view.c:2829]: (style) Local variable 'start' shadows outer variable
[src\p_view.c:2354] -> [src\p_view.c:2830]: (style) Local variable 'up' shadows outer variable
[src\p_view.c:50] -> [src\p_view.c:2831]: (style) Local variable 'forward' shadows outer variable
[src\q_shared.c:1286] -> [src\q_shared.c:1288]: (warning) Opposite inner 'if' condition leads to a dead code block.
[src\q_shared.c:1332] -> [src\q_shared.c:1334]: (warning) Opposite inner 'if' condition leads to a dead code block.
[src\q_shared.c:1400] -> [src\q_shared.c:1376]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[src\q_shared.c:1400] -> [src\q_shared.c:1388]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[src\q_shared.c:1400] -> [src\q_shared.c:1394]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[src\q_shared.c:266]: (portability) Casting from float * to signed int * is not portable due to different binary data representations on different platforms.
[src\q_shared.c:268]: (portability) Casting between signed int * and float * which have an incompatible binary data representation.
[src\q_shared.h:197] -> [src\q_shared.c:291]: (warning) Function 'LerpAngle' argument order different: declaration 'a1, a2, frac' definition 'a2, a1, frac'
[src\q_shared.c:313] -> [src\q_shared.c:47]: (style) Local variable 'i' shadows outer variable
[src\q_shared.c:313] -> [src\q_shared.c:163]: (style) Local variable 'i' shadows outer variable
[src\q_shared.c:313] -> [src\q_shared.c:320]: (style) Local variable 'i' shadows outer variable
[src\q_shared.c:314] -> [src\q_shared.c:323]: (style) Local variable 'corners' shadows outer variable
[src\q_shared.c:313] -> [src\q_shared.c:666]: (style) Local variable 'i' shadows outer variable
[src\q_shared.c:313] -> [src\q_shared.c:772]: (style) Local variable 'i' shadows outer variable
[src\q_shared.c:313] -> [src\q_shared.c:849]: (style) Local variable 'i' shadows outer variable
[src\q_shared.c:313] -> [src\q_shared.c:1171]: (style) Local variable 'i' shadows outer variable
[src\x_fire.c:1182] -> [src\x_fire.c:1184]: (style) Variable 'fireball->s.effects' is reassigned a value before the old one has been used.
[src\ai\ai_class_dmbot.c:1126] -> [src\ai\ai_class_dmbot.c:1264]: (warning) Either the condition 'weapon!=9' is redundant or the array 'AIWeapons[9]' is accessed at index 9, which is out of bounds.
[src\ai\ai_class_dmbot.c:1126] -> [src\ai\ai_class_dmbot.c:1270]: (warning) Either the condition 'weapon!=9' is redundant or the array 'AIWeapons[9]' is accessed at index 9, which is out of bounds.
[src\ai\ai_class_dmbot.c:1126] -> [src\ai\ai_class_dmbot.c:1275]: (warning) Either the condition 'weapon!=9' is redundant or the array 'AIWeapons[9]' is accessed at index 9, which is out of bounds.
[src\ai\ai_class_dmbot.c:438]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[src\ai\ai_class_dmbot.c:451]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[src\ai\ai_class_dmbot.c:984]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[src\ai\ai_class_dmbot.c:1905] -> [src\ai\ai_class_dmbot.c:1906]: (style) Variable 'self->client->ping' is reassigned a value before the old one has been used.
[src\ai\ai_dropnodes.c:353] -> [src\ai\ai_dropnodes.c:357]: (style) Expression is always true because 'else if' condition is opposite to previous condition at line 353.
[src\ai\ai_links.c:178] -> [src\ai\ai_links.c:184]: (warning) Either the condition 'pLinks[n1].numLinks>16' is redundant or the array 'pLinks[n1].nodes[16]' is accessed at index 16, which is out of bounds.
[src\ai\ai_links.c:875] -> [src\ai\ai_links.c:914]: (style) Local variable 'link' shadows outer variable
[src\ai\ai_main.c:1027] -> [src\ai\ai_main.c:1593]: (style) Local variable 'soundfile' shadows outer variable
[src\ai\ai_main.c:1038] -> [src\ai\ai_main.c:1750]: (style) Local variable 'dist' shadows outer variable
[src\ai\ai_navigation.c:333]: (error) Array 'check[3]' accessed at index 3, which is out of bounds.
[src\ai\ai_navigation.c:336] -> [src\ai\ai_navigation.c:363]: (warning) Either the condition 'c<4' is redundant or the array 'check[3]' is accessed at index 3, which is out of bounds.
[src\g_local.h:1033] -> [src\ai\ai_navigation.c:274]: (style) Local variable 'range' shadows outer function
[src\ai\ai_weapons.c:127]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[src\ai\ai_weapons.c:128]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[src\ai\ai_weapons.c:129]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[src\ai\ai_weapons.c:130]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[src\ai\ai_weapons.c:131]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[src\ai\ai_weapons.c:132]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[src\ai\ai_weapons.c:133]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[src\ai\bot_spawn.c:527] -> [src\ai\bot_spawn.c:527]: (warning) Either the condition '!bot->client' is redundant or there is possible null pointer dereference: bot->client.

[MrG-DRGN]

This is specific to yours. It MAY current;y be the same as above.
[g_ai.c:118]: (style) The scope of the variable 'v' can be reduced.
[g_ai.c:434]: (style) The scope of the variable 'r' can be reduced.
[g_ai.c:632]: (style) The scope of the variable 'spot1' can be reduced.
[g_ai.c:632]: (style) The scope of the variable 'spot2' can be reduced.
[g_ai.c:944]: (style) The scope of the variable 'd2' can be reduced.
[g_ai.c:946]: (style) The scope of the variable 'v_forward' can be reduced.
[g_ai.c:946]: (style) The scope of the variable 'v_right' can be reduced.
[g_ai.c:947]: (style) The scope of the variable 'left' can be reduced.
[g_ai.c:947]: (style) The scope of the variable 'center' can be reduced.
[g_ai.c:947]: (style) The scope of the variable 'right' can be reduced.
[g_arty.c:288]: (style) Redundant condition: If 'EXPR == 4', the comparison 'EXPR' is always true.
[g_arty.c:360] -> [g_arty.c:367]: (style) Variable 'check->obj_owner' is reassigned a value before the old one has been used.
[g_arty.c:609] -> [g_arty.c:617]: (style) Variable 'ent->nextthink' is reassigned a value before the old one has been used.
[g_arty.c:1070] -> [g_arty.c:1076]: (style) Variable 'ent->nextthink' is reassigned a value before the old one has been used.
[g_arty.c:430]: (style) Variable 'tr' is assigned a value that is never used.
[g_combat.c:306]: (style) Assignment of function parameter has no effect outside the function.
[g_combat.c:793]: (style) Redundant condition: If 'EXPR == 58', the comparison 'EXPR' is always true.
[g_combat.c:1650] -> [g_combat.c:1652]: (style) Variable 'speed' is reassigned a value before the old one has been used.
[g_combat.c:433]: (style) The scope of the variable 'x_coord' can be reduced.
[g_combat.c:433]: (style) The scope of the variable 'y_coord' can be reduced.
[g_combat.c:433]: (style) The scope of the variable 'p_angle' can be reduced.
[g_combat.c:761]: (style) The scope of the variable 'randnum' can be reduced.
[g_combat.c:767]: (style) The scope of the variable 'i' can be reduced.
[g_combat.c:768]: (style) The scope of the variable 'v' can be reduced.
[g_combat.c:772]: (style) The scope of the variable 'check_ent' can be reduced.
[g_combat.c:1543]: (style) The scope of the variable 'speed' can be reduced.
[g_combat.c:306]: (style) Variable 'damage' is assigned a value that is never used.
[g_combat.c:990]: (style) Variable 'damage' is assigned a value that is never used.
[g_combat.c:991]: (style) Variable 'result' is assigned a value that is never used.
[g_combat.c:1346]: (style) Variable 'save' is assigned a value that is never used.
[g_cmds.c:343]: (style) Redundant condition: If '!EXPR', the comparison 'EXPR != 120' is always true.
[g_cmds.c:1354]: (style) Redundant condition: it. '!it || (it && !it->use)' is equivalent to '!it || !it->use'
[g_cmds.c:3015]: (style) Redundant condition: If 'EXPR == 5', the comparison 'EXPR' is always true.
[g_cmds.c:1354] -> [g_cmds.c:1356]: (warning) Either the condition '!it' is redundant or there is possible null pointer dereference: it.
[g_cmds.c:3259]: (style) Same expression on both sides of '||'.
[g_cmds.c:334]: (style) The scope of the variable 'e' can be reduced.
[g_cmds.c:732]: (style) The scope of the variable 'index' can be reduced.
[g_cmds.c:777]: (style) The scope of the variable 'index' can be reduced.
[g_cmds.c:846]: (style) The scope of the variable 'it_ent' can be reduced.
[g_cmds.c:1492]: (style) The scope of the variable 'dropped' can be reduced.
[g_cmds.c:1604]: (style) The scope of the variable 'entryb' can be reduced.
[g_cmds.c:1771]: (style) The scope of the variable 'index' can be reduced.
[g_cmds.c:1825]: (style) The scope of the variable 'index' can be reduced.
[g_cmds.c:2411]: (style) The scope of the variable 'i' can be reduced.
[g_cmds.c:2411]: (style) The scope of the variable 'offset_of_text' can be reduced.
[g_cmds.c:2999]: (style) The scope of the variable 'ammo_index' can be reduced.
[g_cmds.c:3125]: (style) The scope of the variable 'start' can be reduced.
[g_cmds.c:3126]: (style) The scope of the variable 'end' can be reduced.
[g_cmds.c:3127]: (style) The scope of the variable 'up' can be reduced.
[g_cmds.c:3363]: (style) The scope of the variable 'motd' can be reduced.
[g_cmds.c:3364]: (style) The scope of the variable 'line' can be reduced.
[g_cmds.c:3444]: (style) The scope of the variable 'tmp' can be reduced.
[g_cmds.c:3056]: (error) Uninitialized variable: ammo_amount
[g_cmds.c:55]: (style) Variable 'pw' is assigned a value that is never used.
[g_cmds.c:355]: (style) Variable 'weight' is assigned a value that is never used.
[g_ents.c:426]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:441]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:456]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:472]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:524]: (warning) Found calculation inside sizeof().
[g_func.c:423] -> [g_func.c:421]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_func.c:1146] -> [g_func.c:1144]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_func.c:1547] -> [g_func.c:1545]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_func.c:2067] -> [g_func.c:2065]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_func.c:476] -> [g_func.c:482]: (style) Variable 'tmin[2]' is reassigned a value before the old one has been used.
[g_func.c:1046]: (style) The scope of the variable 'newspeed' can be reduced.
[g_func.c:1047]: (style) The scope of the variable 'ratio' can be reduced.
[g_func.c:1048]: (style) The scope of the variable 'dist' can be reduced.
[g_items.c:794]: (style) Redundant condition: If 'EXPR == 1', the comparison 'EXPR' is always true.
[g_items.c:557]: (style) The scope of the variable 'grennum' can be reduced.
[g_items.c:696]: (style) The scope of the variable 'index' can be reduced.
[g_items.c:753]: (style) The scope of the variable 'vol' can be reduced.
[g_items.c:896]: (style) The scope of the variable 'offset' can be reduced.
[g_items.c:1086]: (style) The scope of the variable 'start' can be reduced.
[g_items.c:1088]: (style) The scope of the variable 'len' can be reduced.
[g_items.c:1089]: (style) The scope of the variable 'ammo' can be reduced.
[g_items.c:1684]: (style) The scope of the variable 'it' can be reduced.
[g_items.c:567]: (style) Variable 'grennum' is assigned a value that is never used.
[g_maps.c:118]: (warning) fscanf() without field width limits can crash with huge input data.
[g_maps.c:135]: (warning) fscanf() without field width limits can crash with huge input data.
[g_maps.c:108]: (style) The scope of the variable 'i' can be reduced.
[g_maps.c:109]: (style) The scope of the variable 'szLineIn' can be reduced.
[g_maps.c:355]: (style) The scope of the variable 'i' can be reduced.
[g_maps.c:357]: (style) The scope of the variable 'ent' can be reduced.
[g_main.c:399]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:436]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:667]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:703]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:805]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:713] -> [g_main.c:712] -> [g_main.c:736] -> [g_main.c:738] -> [g_main.c:341]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[g_main.c:713] -> [g_main.c:712] -> [g_main.c:736] -> [g_main.c:738] -> [g_main.c:351]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[g_main.c:713] -> [g_main.c:712] -> [g_main.c:736] -> [g_main.c:738] -> [g_main.c:360]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[g_main.c:713] -> [g_main.c:736] -> [g_main.c:737]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: check.
[g_main.c:737]: (error) Null pointer dereference
[g_main.c:201]: (style) The scope of the variable 'ent' can be reduced.
[g_main.c:299]: (style) The scope of the variable 'ent' can be reduced.
[g_main.c:391]: (style) The scope of the variable 'i' can be reduced.
[g_main.c:391]: (style) The scope of the variable 'c' can be reduced.
[g_main.c:392]: (style) The scope of the variable 'f' can be reduced.
[g_main.c:416]: (style) The scope of the variable 'c' can be reduced.
[g_main.c:418]: (style) The scope of the variable 'f' can be reduced.
[g_main.c:523]: (style) The scope of the variable 's' can be reduced.
[g_main.c:523]: (style) The scope of the variable 'f' can be reduced.
[g_main.c:523]: (style) The scope of the variable 'sb' can be reduced.
[g_main.c:524]: (style) The scope of the variable 'seps' can be reduced.
[g_main.c:525]: (style) The scope of the variable 'check' can be reduced.
[g_main.c:526]: (style) The scope of the variable 'axiscount' can be reduced.
[g_main.c:526]: (style) The scope of the variable 'alliedcount' can be reduced.
[g_main.c:921]: (style) The scope of the variable 'tempscore' can be reduced.
[g_main.c:924]: (style) The scope of the variable 'cl' can be reduced.
[g_main.c:926]: (style) The scope of the variable 'allies_kills_win' can be reduced.
[g_main.c:927]: (style) The scope of the variable 'axis_kills_win' can be reduced.
[g_main.c:928]: (style) The scope of the variable 'allies_points_win' can be reduced.
[g_main.c:929]: (style) The scope of the variable 'axis_points_win' can be reduced.
[g_main.c:942]: (style) The scope of the variable 'w' can be reduced.
[g_main.c:943]: (style) The scope of the variable 'range' can be reduced.
[g_main.c:945]: (style) The scope of the variable 'usaflag' can be reduced.
[g_main.c:946]: (style) The scope of the variable 'grmflag' can be reduced.
[g_main.c:1240]: (style) The scope of the variable 'campaignfilename' can be reduced.
[g_main.c:1295]: (style) The scope of the variable 'check_ent' can be reduced.
[g_local.h:1034] -> [g_main.c:943]: (style) Local variable 'range' shadows outer function
[g_main.c:398]: (style) Variable 'c' is assigned a value that is never used.
[g_main.c:805]: (style) Variable 'sb' is assigned a value that is never used.
[g_main.c:805]: (style) Variable 'sb' is allocated memory that is never used.
[g_monster.c:158]: (style) The scope of the variable 'dmg' can be reduced.
[g_monster.c:539]: (style) The scope of the variable 'v' can be reduced.
[g_misc.c:256] -> [g_misc.c:264]: (style) Variable 'gib->solid' is reassigned a value before the old one has been used.
[g_misc.c:1798]: (warning) Redundant assignment of 'ent->s.frame' to itself.
[g_misc.c:1822]: (warning) Redundant assignment of 'ent->s.frame' to itself.
[g_misc.c:1885]: (warning) Redundant assignment of 'ent->s.frame' to itself.
[g_misc.c:661]: (style) The scope of the variable 'activator' can be reduced.
[g_misc.c:1880]: (style) The scope of the variable 'i' can be reduced.
[g_objectives.c:57]: (style) The scope of the variable 'delay' can be reduced.
[g_objectives.c:166]: (style) The scope of the variable 'otherteam' can be reduced.
[g_objectives.c:373]: (style) The scope of the variable 'otherteam' can be reduced.
[g_objectives.c:850]: (style) The scope of the variable 'e' can be reduced.
[g_objectives.c:851]: (style) The scope of the variable 'i' can be reduced.
[g_objectives.c:791]: (style) Variable 'item' is assigned a value that is never used.
[g_phys.c:156]: (style) The scope of the variable 'change' can be reduced.
[g_phys.c:684]: (style) The scope of the variable 'backoff' can be reduced.
[g_phys.c:834]: (style) The scope of the variable 'vel' can be reduced.
[g_phys.c:838]: (style) The scope of the variable 'mask' can be reduced.
[g_save.c:869] -> [g_save.c:877]: (error) Resource handle 'f' freed twice.
[g_save.c:474]: (style) The scope of the variable 'len' can be reduced.
[g_save.c:791]: (style) The scope of the variable 'ent' can be reduced.
[g_spawn.c:482]: (style) Array index 'i' is used before limits check.
[g_spawn.c:926]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_spawn.c:531]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:532]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:533]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:539]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:543]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:544]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:545]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:568]: (style) The scope of the variable 'com_token' can be reduced.
[g_spawn.c:674]: (style) The scope of the variable 'com_token' can be reduced.
[g_spawn.c:895]: (style) The scope of the variable 'c' can be reduced.
[g_spawn.c:897]: (style) The scope of the variable 's' can be reduced.
[g_spawn.c:897]: (style) The scope of the variable 'f' can be reduced.
[g_spawn.c:901]: (style) The scope of the variable 'x' can be reduced.
[g_spawn.c:901]: (style) The scope of the variable 'y' can be reduced.
[g_spawn.c:901]: (style) The scope of the variable 'z' can be reduced.
[g_spawn.c:902]: (style) The scope of the variable 'angle' can be reduced.
[g_spawn.c:903]: (style) The scope of the variable 'team' can be reduced.
[g_spawn.c:904]: (style) The scope of the variable 'stance' can be reduced.
[g_spawn.c:1011]: (style) The scope of the variable 'com_token' can be reduced.
[g_spawn.c:793]: (style) Variable 'i' is assigned a value that is never used.
[g_svcmds.c:444]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_svcmds.c:78] -> [g_svcmds.c:80]: (style) Variable 'ent->client->resp.mos' is reassigned a value before the old one has been used.
[g_svcmds.c:403]: (style) The scope of the variable 'c' can be reduced.
[g_svcmds.c:404]: (style) The scope of the variable 'f' can be reduced.
[g_svcmds.c:409]: (style) The scope of the variable 'owner' can be reduced.
[g_svcmds.c:410]: (style) The scope of the variable 'xpos' can be reduced.
[g_svcmds.c:410]: (style) The scope of the variable 'ypos' can be reduced.
[g_svcmds.c:426]: (style) Variable 'check' is assigned a value that is never used.
[g_target.c:68]: (style) The scope of the variable 'chan' can be reduced.
[g_target.c:526]: (style) The scope of the variable 'last_movedir' can be reduced.
[g_target.c:445]: (style) Variable 'effect' is assigned a value that is never used.
[g_target.c:447]: (style) Variable 'effect' is assigned a value that is never used.
[g_target.c:449]: (style) Variable 'effect' is assigned a value that is never used.
[g_turret.c:922]: (style) Redundant condition: If 'EXPR == 1', the comparison 'EXPR' is always true.
[g_turret.c:187]: (warning) Comparison of modulo result is predetermined, because it is always less than 2.
[g_turret.c:106] -> [g_turret.c:103]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_turret.c:936] -> [g_turret.c:956]: (style) Variable 'self->movetype' is reassigned a value before the old one has been used.
[g_turret.c:282]: (warning) Redundant assignment of 'shell->nextthink' to itself.
[g_turret.c:93]: (style) The scope of the variable 'attacker' can be reduced.
[g_turret.c:406]: (style) Variable 'speed' is assigned a value that is never used.
[g_turret.c:408]: (style) Variable 'speed' is assigned a value that is never used.
[g_turretx.c:859]: (style) Redundant condition: If 'EXPR == 1', the comparison 'EXPR' is always true.
[g_turretx.c:106] -> [g_turretx.c:103]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_turretx.c:263]: (warning) Redundant assignment of 'shell->nextthink' to itself.
[g_turretx.c:93]: (style) The scope of the variable 'attacker' can be reduced.
[g_turretx.c:385]: (style) Variable 'speed' is assigned a value that is never used.
[g_turretx.c:387]: (style) Variable 'speed' is assigned a value that is never used.
[g_utils.c:410]: (style) The scope of the variable 'forward' can be reduced.
[g_utils.c:526]: (style) The scope of the variable 'hit' can be reduced.
[g_utils.c:562]: (style) The scope of the variable 'hit' can be reduced.
[g_utils.c:759]: (style) The scope of the variable 'ent' can be reduced.
[m_actor.c:519]: (style) The scope of the variable 'ent' can be reduced.
[m_civilian.c:166] -> [m_civilian.c:183]: (style) Variable 'tr' is reassigned a value before the old one has been used.
[m_civilian.c:303]: (style) The scope of the variable 'n' can be reduced.
[m_civilian.c:332]: (style) The scope of the variable 'v' can be reduced.
[m_civilian.c:333]: (style) The scope of the variable 'i' can be reduced.
[m_civilian.c:335]: (style) The scope of the variable 'temp_distance' can be reduced.
[m_civilian.c:335]: (style) The scope of the variable 'nearest_distance' can be reduced.
[m_civilian.c:656]: (style) The scope of the variable 'cl_ent' can be reduced.
[m_civilian.c:657]: (style) The scope of the variable 'j' can be reduced.
[m_civilian.c:334] -> [m_civilian.c:381]: (style) Local variable 'e' shadows outer variable
[m_civilian.c:334] -> [m_civilian.c:381]: (style) Local variable 'nearest' shadows outer variable
[m_civilian.c:335] -> [m_civilian.c:382]: (style) Local variable 'temp_distance' shadows outer variable
[m_civilian.c:335] -> [m_civilian.c:382]: (style) Local variable 'nearest_distance' shadows outer variable
[g_weapon.c:586]: (style) Boolean result is used in bitwise operation. Clarify expression with parentheses.
[g_weapon.c:2035]: (style) Redundant condition: ent->owner. '!ent->owner || (ent->owner && !ent->owner->inuse)' is equivalent to '!ent->owner || !ent->owner->inuse'
[g_weapon.c:4310]: (style) Redundant condition: ent->owner->client->resp.team_on. '!A || (A && B)' is equivalent to '!A || B'
[g_weapon.c:1038] -> [g_weapon.c:1039]: (style) Variable 'underwater_bullet->think' is reassigned a value before the old one has been used.
[g_weapon.c:2157] -> [g_weapon.c:2165]: (style) Variable 'rocket->think' is reassigned a value before the old one has been used.
[g_weapon.c:2217] -> [g_weapon.c:2225]: (style) Variable 'rocket->think' is reassigned a value before the old one has been used.
[g_weapon.c:5021] -> [g_weapon.c:5027]: (style) Variable 'ent->client->kick_angles[0]' is reassigned a value before the old one has been used.
[g_weapon.c:5208] -> [g_weapon.c:5214]: (style) Variable 'ent->client->kick_angles[0]' is reassigned a value before the old one has been used.
[g_weapon.c:6170] -> [g_weapon.c:6176]: (style) Variable 'ent->client->kick_angles[0]' is reassigned a value before the old one has been used.
[g_weapon.c:2081]: (warning) Redundant assignment of 'rocket->nextthink' to itself.
[g_weapon.c:2171]: (warning) Redundant assignment of 'rocket->nextthink' to itself.
[g_weapon.c:2231]: (warning) Redundant assignment of 'rocket->nextthink' to itself.
[g_weapon.c:68]: (style) The scope of the variable 'v' can be reduced.
[g_weapon.c:70]: (style) The scope of the variable 'eta' can be reduced.
[g_weapon.c:350]: (style) The scope of the variable 'forward' can be reduced.
[g_weapon.c:350]: (style) The scope of the variable 'right' can be reduced.
[g_weapon.c:350]: (style) The scope of the variable 'up' can be reduced.
[g_weapon.c:352]: (style) The scope of the variable 'r' can be reduced.
[g_weapon.c:353]: (style) The scope of the variable 'u' can be reduced.
[g_weapon.c:493]: (style) The scope of the variable 'color' can be reduced.
[g_weapon.c:672]: (style) The scope of the variable 'forward' can be reduced.
[g_weapon.c:672]: (style) The scope of the variable 'right' can be reduced.
[g_weapon.c:672]: (style) The scope of the variable 'up' can be reduced.
[g_weapon.c:674]: (style) The scope of the variable 'r' can be reduced.
[g_weapon.c:675]: (style) The scope of the variable 'u' can be reduced.
[g_weapon.c:718]: (style) The scope of the variable 'color' can be reduced.
[g_weapon.c:1127]: (style) The scope of the variable 'mod' can be reduced.
[g_weapon.c:1466]: (style) The scope of the variable 'cl_ent' can be reduced.
[g_weapon.c:1699]: (style) The scope of the variable 'grenadefilename' can be reduced.
[g_weapon.c:3144]: (style) The scope of the variable 'up' can be reduced.
[g_weapon.c:3311]: (style) The scope of the variable 'kick' can be reduced.
[g_weapon.c:3317]: (style) The scope of the variable 'end' can be reduced.
[g_weapon.c:3318]: (style) The scope of the variable 'g_offset' can be reduced.
[g_weapon.c:3553]: (style) The scope of the variable 'kick' can be reduced.
[g_weapon.c:3559]: (style) The scope of the variable 'end' can be reduced.
[g_weapon.c:3560]: (style) The scope of the variable 'g_offset' can be reduced.
[g_weapon.c:4299]: (style) The scope of the variable 'randnum' can be reduced.
[g_weapon.c:4300]: (style) The scope of the variable 'inwater' can be reduced.
[g_weapon.c:4444]: (style) The scope of the variable 'end' can be reduced.
[g_weapon.c:4445]: (style) The scope of the variable 'g_offset' can be reduced.
[g_weapon.c:5172]: (style) The scope of the variable 'end' can be reduced.
[g_weapon.c:5276]: (style) The scope of the variable 'forward' can be reduced.
[g_weapon.c:5276]: (style) The scope of the variable 'right' can be reduced.
[g_weapon.c:5276]: (style) The scope of the variable 'up' can be reduced.
[g_weapon.c:5278]: (style) The scope of the variable 'r' can be reduced.
[g_weapon.c:5279]: (style) The scope of the variable 'u' can be reduced.
[g_weapon.c:5330]: (style) The scope of the variable 'color' can be reduced.
[g_weapon.c:5495]: (style) The scope of the variable 'dot' can be reduced.
[g_weapon.c:5940]: (style) The scope of the variable 'i' can be reduced.
[g_weapon.c:6248]: (style) The scope of the variable 'end' can be reduced.
[g_weapon.c:6249]: (style) The scope of the variable 'g_offset' can be reduced.
[g_weapon.c:6440]: (style) The scope of the variable 'dot' can be reduced.
[g_weapon.c:6572]: (style) The scope of the variable 'startc' can be reduced.
[g_weapon.c:6573]: (style) The scope of the variable 'forwardc' can be reduced.
[g_weapon.c:6573]: (style) The scope of the variable 'rightc' can be reduced.
[g_weapon.c:6577]: (style) The scope of the variable 'endc' can be reduced.
[g_weapon.c:6578]: (style) The scope of the variable 'g_offset' can be reduced.
[g_local.h:1034] -> [g_weapon.c:102]: (style) Local variable 'range' shadows outer function
[g_weapon.c:1349] -> [g_weapon.c:1410]: (style) Local variable 'i' shadows outer variable
[g_weapon.c:1349] -> [g_weapon.c:1465]: (style) Local variable 'i' shadows outer variable
[g_weapon.c:284]: (style) Variable 'water' is assigned a value that is never used.
[g_weapon.c:293]: (style) Variable 'water' is assigned a value that is never used.
[g_weapon.c:567]: (style) Variable 'hit_ent' is assigned a value that is never used.
[g_weapon.c:923]: (style) Variable 'effect' is assigned a value that is never used.
[g_weapon.c:993]: (style) Variable 'effect' is assigned a value that is never used.
[g_weapon.c:1066]: (style) Variable 'effect' is assigned a value that is never used.
[g_weapon.c:1619]: (style) Variable 'index' is assigned a value that is never used.
[g_weapon.c:3548]: (style) Variable 'shots' is assigned a value that is never used.
[g_weapon.c:3840]: (style) Variable 'mod' is assigned a value that is never used.
[g_weapon.c:3841]: (style) Variable 'radius_damage' is assigned a value that is never used.
[g_weapon.c:4433]: (style) Variable 'shots' is assigned a value that is never used.
[g_weapon.c:4700]: (style) Variable 'mod' is assigned a value that is never used.
[g_weapon.c:5443]: (style) Variable 'mod' is assigned a value that is never used.
[g_weapon.c:5888]: (style) Variable 'mag_index' is assigned a value that is never used.
[g_weapon.c:6237]: (style) Variable 'shots' is assigned a value that is never used.
[g_weapon.c:6747]: (style) Variable 'mod' is assigned a value that is never used.
[m_insane.c:591]: (style) The scope of the variable 'n' can be reduced.
[m_move.c:122]: (style) The scope of the variable 'dz' can be reduced.
[m_move.c:125]: (style) The scope of the variable 'i' can be reduced.
[m_move.c:364]: (style) The scope of the variable 'delta' can be reduced.
[p_classes.c:171] -> [p_classes.c:168] -> [p_classes.c:42]: (warning) Either the condition '!item' is redundant or there is possible null pointer dereference: item.
[p_generic_wep.c:420]: (style) Variable 'ammo_item' is assigned a value that is never used.
[p_client.c:2964]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:3388]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:4648]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:4891]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:1990] -> [p_client.c:1999]: (style) Condition '!spot' is always true
[p_client.c:2039] -> [p_client.c:2059]: (style) Condition '!spot' is always true
[p_client.c:4850]: (error) Memory leak: f
[p_client.c:742] -> [p_client.c:733]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[p_client.c:742] -> [p_client.c:734]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[p_client.c:742] -> [p_client.c:736]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[p_client.c:742] -> [p_client.c:737]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[p_client.c:2822] -> [p_client.c:2815]: (warning) Either the condition 'if(ent)' is redundant or there is possible null pointer dereference: ent.
[p_client.c:2822] -> [p_client.c:2921]: (warning) Either the condition 'ent' is redundant or there is possible null pointer dereference: ent.
[p_client.c:2163] -> [p_client.c:2167]: (style) Variable 'body->svflags' is reassigned a value before the old one has been used.
[p_client.c:4248] -> [p_client.c:4250]: (style) Variable 'temp_damage' is reassigned a value before the old one has been used.
[p_client.c:97]: (style) The scope of the variable 'spot' can be reduced.
[p_client.c:345]: (style) The scope of the variable 'mod' can be reduced.
[p_client.c:345]: (style) The scope of the variable 'i' can be reduced.
[p_client.c:346]: (style) The scope of the variable 'message' can be reduced.
[p_client.c:347]: (style) The scope of the variable 'message2' can be reduced.
[p_client.c:348]: (style) The scope of the variable 'message3' can be reduced.
[p_client.c:351]: (style) The scope of the variable 'Tent' can be reduced.
[p_client.c:978]: (style) The scope of the variable 'dropped' can be reduced.
[p_client.c:1226]: (style) The scope of the variable 'n' can be reduced.
[p_client.c:1227]: (style) The scope of the variable 'forward' can be reduced.
[p_client.c:1227]: (style) The scope of the variable 'right' can be reduced.
[p_client.c:1583]: (style) The scope of the variable 'player' can be reduced.
[p_client.c:1625]: (style) The scope of the variable 'range' can be reduced.
[p_client.c:1688]: (style) The scope of the variable 'e' can be reduced.
[p_client.c:1800]: (style) The scope of the variable 'check_ent' can be reduced.
[p_client.c:1900]: (style) The scope of the variable 'bestplayerdistance' can be reduced.
[p_client.c:1949]: (style) The scope of the variable 'target' can be reduced.
[p_client.c:2097]: (style) The scope of the variable 'ent' can be reduced.
[p_client.c:2715]: (style) The scope of the variable 'i' can be reduced.
[p_client.c:3364]: (style) The scope of the variable 'c' can be reduced.
[p_client.c:3366]: (style) The scope of the variable 'f' can be reduced.
[p_client.c:3540]: (style) The scope of the variable 'other' can be reduced.
[p_client.c:3550]: (style) The scope of the variable 'cmd' can be reduced.
[p_client.c:3557]: (style) The scope of the variable 'time' can be reduced.
[p_client.c:4587]: (style) The scope of the variable 'c' can be reduced.
[p_client.c:4590]: (style) The scope of the variable 'f' can be reduced.
[p_client.c:4594]: (style) The scope of the variable 'name' can be reduced.
[p_client.c:4596]: (style) The scope of the variable 'ping' can be reduced.
[p_client.c:4617]: (style) The scope of the variable 'chat' can be reduced.
[p_client.c:4856]: (style) The scope of the variable 'c' can be reduced.
[p_client.c:4858]: (style) The scope of the variable 'f' can be reduced.
[p_client.c:4862]: (style) The scope of the variable 'name' can be reduced.
[p_client.c:4864]: (style) The scope of the variable 'games' can be reduced.
[p_client.c:4865]: (style) The scope of the variable 'ping' can be reduced.
[p_client.c:4871]: (style) The scope of the variable 'i' can be reduced.
[p_client.c:1226] -> [p_client.c:1343]: (style) Local variable 'i' shadows outer variable
[g_local.h:1034] -> [p_client.c:1625]: (style) Local variable 'range' shadows outer function
[p_client.c:1764]: (style) Variable 'range1' is assigned a value that is never used.
[p_client.c:1764]: (style) Variable 'range2' is assigned a value that is never used.
[p_client.c:1765]: (style) Variable 'spot1' is assigned a value that is never used.
[p_client.c:4596]: (style) Variable 'ping' is assigned a value that is never used.
[p_client.c:4647]: (style) Variable 'c' is assigned a value that is never used.
[p_client.c:4652]: (style) Variable 'name' is assigned a value that is never used.
[p_client.c:4660]: (style) Variable 'ping' is assigned a value that is never used.
[p_client.c:4744]: (style) Variable 'chat' is assigned a value that is never used.
[p_client.c:4863]: (style) Variable 'ratio' is assigned a value that is never used.
[p_client.c:4864]: (style) Variable 'games' is assigned a value that is never used.
[p_client.c:4865]: (style) Variable 'ping' is assigned a value that is never used.
[p_client.c:4890]: (style) Variable 'c' is assigned a value that is never used.
[p_client.c:4896]: (style) Variable 'name' is assigned a value that is never used.
[p_client.c:4900]: (style) Variable 'games' is assigned a value that is never used.
[p_client.c:4905]: (style) Variable 'ping' is assigned a value that is never used.
[p_hud.c:793]: (error) Buffer is accessed out of bounds: pingstring
[p_hud.c:795]: (error) Buffer is accessed out of bounds: pingstring
[p_hud.c:801]: (error) Buffer is accessed out of bounds: pingstring
[p_hud.c:841]: (error) Buffer is accessed out of bounds: pingstring
[p_hud.c:843]: (error) Buffer is accessed out of bounds: pingstring
[p_hud.c:849]: (error) Buffer is accessed out of bounds: pingstring
[p_hud.c:180]: (style) Boolean result is used in bitwise operation. Clarify expression with parentheses.
[p_hud.c:149]: (style) The scope of the variable 'n' can be reduced.
[p_hud.c:604]: (style) The scope of the variable 'scoreleftpic' can be reduced.
[p_hud.c:605]: (style) The scope of the variable 'scorerightpic' can be reduced.
[p_hud.c:606]: (style) The scope of the variable 'scoretopleftpic' can be reduced.
[p_hud.c:607]: (style) The scope of the variable 'scoretoprightpic' can be reduced.
[p_hud.c:610]: (style) The scope of the variable 'cl' can be reduced.
[p_hud.c:932]: (style) The scope of the variable 'scoreleftpic' can be reduced.
[p_hud.c:933]: (style) The scope of the variable 'scorerightpic' can be reduced.
[p_hud.c:934]: (style) The scope of the variable 'scoretopleftpic' can be reduced.
[p_hud.c:935]: (style) The scope of the variable 'scoretoprightpic' can be reduced.
[p_hud.c:938]: (style) The scope of the variable 'cl' can be reduced.
[p_hud.c:1572]: (style) The scope of the variable 'item' can be reduced.
[p_hud.c:1573]: (style) The scope of the variable 'index' can be reduced.
[p_hud.c:1419]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1422]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1426]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1429]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1431]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1440]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1444]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1471]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:786]: (style) Variable 'cl' is assigned a value that is never used.
[p_hud.c:834]: (style) Variable 'cl' is assigned a value that is never used.
[p_hud.c:1107]: (style) Variable 'cl' is assigned a value that is never used.
[p_hud.c:1159]: (style) Variable 'cl' is assigned a value that is never used.
[p_menu.c:33]: (style) The scope of the variable 'p' can be reduced.
[p_observer.c:267] -> [p_observer.c:277]: (style) Variable 'tr' is reassigned a value before the old one has been used.
[p_observer.c:876] -> [p_observer.c:887]: (style) Variable 'i' is reassigned a value before the old one has been used.
[p_observer.c:987] -> [p_observer.c:989]: (style) Variable 'ent->client->resp.mos' is reassigned a value before the old one has been used.
[p_observer.c:43]: (style) The scope of the variable 'check_ent' can be reduced.
[p_observer.c:454]: (style) The scope of the variable 'cl_ent' can be reduced.
[p_observer.c:715]: (style) The scope of the variable 'taken' can be reduced.
[p_observer.c:716]: (style) The scope of the variable 'maxSlots' can be reduced.
[p_observer.c:866]: (style) The scope of the variable 'i' can be reduced.
[p_observer.c:866]: (style) The scope of the variable 'j' can be reduced.
[p_observer.c:866]: (style) The scope of the variable 'k' can be reduced.
[p_observer.c:866]: (style) The scope of the variable 'l' can be reduced.
[p_observer.c:866]: (style) The scope of the variable 'm' can be reduced.
[p_observer.c:1011]: (style) The scope of the variable 'i' can be reduced.
[p_observer.c:1013]: (style) The scope of the variable 'max_clients' can be reduced.
[p_observer.c:554]: (style) Variable 'spots' is assigned a value that is never used.
[p_observer.c:865]: (style) Variable 'foundspot' is assigned a value that is never used.
[p_observer.c:1424]: (style) Variable 'botmap' is assigned a value that is never used.
[p_observer.c:1445]: (style) Variable 'botmap' is assigned a value that is never used.
[p_weapon.c:2390]: (error) Array 'fire_frames[2]' accessed at index 2, which is out of bounds.
[p_weapon.c:1361]: (style) Boolean result is used in bitwise operation. Clarify expression with parentheses.
[p_weapon.c:1216] -> [p_weapon.c:1222]: (style) The if condition is the same as the previous if condition
[p_weapon.c:1502] -> [p_weapon.c:1556]: (warning) Either the condition 'surf' is redundant or there is possible null pointer dereference: surf.
[p_weapon.c:2053] -> [p_weapon.c:2048]: (warning) Either the condition 'if(ent)' is redundant or there is possible null pointer dereference: ent.
[p_weapon.c:922] -> [p_weapon.c:925]: (style) Variable 'speed' is reassigned a value before the old one has been used.
[p_weapon.c:2684] -> [p_weapon.c:2686]: (style) Variable 'ent->client->ps.gunframe' is reassigned a value before the old one has been used.
[p_weapon.c:634]: (style) The scope of the variable 'ammo_index' can be reduced.
[p_weapon.c:635]: (style) The scope of the variable 'ammo_item' can be reduced.
[p_weapon.c:636]: (style) The scope of the variable 'item_rounds' can be reduced.
[p_weapon.c:1462]: (style) The scope of the variable 'dropped' can be reduced.
[p_weapon.c:1785]: (style) The scope of the variable 'knives' can be reduced.
[p_weapon.c:1940]: (style) The scope of the variable 'world_up' can be reduced.
[p_weapon.c:2248]: (style) The scope of the variable 'randnum' can be reduced.
[p_weapon.c:649]: (style) Variable 'ammo_index' is assigned a value that is never used.
[p_weapon.c:652]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:654]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:656]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:658]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:660]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:662]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:664]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:666]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:668]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:670]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:672]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:674]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:676]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:678]: (style) Variable 'item_rounds' is assigned a value that is never used.
[p_weapon.c:1260]: (style) Variable 'radius' is assigned a value that is never used.
[p_weapon.c:1460]: (style) Variable 'fistarmed' is assigned a value that is never used.
[p_weapon.c:1636]: (style) Variable 'item' is assigned a value that is never used.
[p_weapon.c:1715]: (style) Variable 'effect' is assigned a value that is never used.
[p_view.c:858]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[p_view.c:884] -> [p_view.c:888]: (style) The if condition is the same as the previous if condition
[p_view.c:2231] -> [p_view.c:2235]: (style) Variable 'client->anim_end' is reassigned a value before the old one has been used.
[p_view.c:2429] -> [p_view.c:2436]: (style) Variable 'ent->viewheight' is reassigned a value before the old one has been used.
[p_view.c:2873] -> [p_view.c:2874]: (style) Variable 'dist' is reassigned a value before the old one has been used.
[p_view.c:95]: (style) The scope of the variable 'side' can be reduced.
[p_view.c:98]: (style) The scope of the variable 'r' can be reduced.
[p_view.c:98]: (style) The scope of the variable 'l' can be reduced.
[p_view.c:99]: (style) The scope of the variable 'power_color' can be reduced.
[p_view.c:100]: (style) The scope of the variable 'acolor' can be reduced.
[p_view.c:101]: (style) The scope of the variable 'bcolor' can be reduced.
[p_view.c:104]: (style) The scope of the variable 'randnum' can be reduced.
[p_view.c:473]: (style) The scope of the variable 'delta' can be reduced.
[p_view.c:478]: (style) The scope of the variable 'f' can be reduced.
[p_view.c:478]: (style) The scope of the variable 'r' can be reduced.
[p_view.c:478]: (style) The scope of the variable 'u' can be reduced.
[p_view.c:1102]: (style) The scope of the variable 'damage' can be reduced.
[p_view.c:1103]: (style) The scope of the variable 'dir' can be reduced.
[p_view.c:1338]: (style) The scope of the variable 'randnum' can be reduced.
[p_view.c:1555]: (style) The scope of the variable 'pa_type' can be reduced.
[p_view.c:2320]: (style) The scope of the variable 'other' can be reduced.
[p_view.c:2356]: (style) The scope of the variable 'start' can be reduced.
[p_view.c:2357]: (style) The scope of the variable 'end' can be reduced.
[p_view.c:2361]: (style) The scope of the variable 'offset' can be reduced.
[p_view.c:2840]: (style) The scope of the variable 'transition_dir' can be reduced.
[p_view.c:50] -> [p_view.c:1239]: (style) Local variable 'forward' shadows outer variable
[p_view.c:1629] -> [p_view.c:1642]: (style) Local variable 'end' shadows outer variable
[p_view.c:1630] -> [p_view.c:1643]: (style) Local variable 'tr' shadows outer variable
[p_view.c:50] -> [p_view.c:2358]: (style) Local variable 'up' shadows outer variable
[p_view.c:50] -> [p_view.c:2361]: (style) Local variable 'right' shadows outer variable
[p_view.c:50] -> [p_view.c:2692]: (style) Local variable 'forward' shadows outer variable
[p_view.c:2359] -> [p_view.c:2832]: (style) Local variable 'tr' shadows outer variable
[p_view.c:2357] -> [p_view.c:2833]: (style) Local variable 'end' shadows outer variable
[p_view.c:2356] -> [p_view.c:2833]: (style) Local variable 'start' shadows outer variable
[p_view.c:2358] -> [p_view.c:2834]: (style) Local variable 'up' shadows outer variable
[p_view.c:50] -> [p_view.c:2835]: (style) Local variable 'forward' shadows outer variable
[p_view.c:1633]: (style) Variable 'tr' is assigned a value that is never used.
[q_shared.c:1286] -> [q_shared.c:1288]: (warning) Opposite inner 'if' condition leads to a dead code block.
[q_shared.c:1332] -> [q_shared.c:1334]: (warning) Opposite inner 'if' condition leads to a dead code block.
[q_shared.c:1400] -> [q_shared.c:1376]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[q_shared.c:1400] -> [q_shared.c:1388]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[q_shared.c:1400] -> [q_shared.c:1394]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[q_shared.c:266]: (portability) Casting from float * to signed int * is not portable due to different binary data representations on different platforms.
[q_shared.c:268]: (portability) Casting between signed int * and float * which have an incompatible binary data representation.
[q_shared.c:667]: (style) The scope of the variable 'val' can be reduced.
[q_shared.c:691]: (style) The scope of the variable 'ilength' can be reduced.
[q_shared.c:710]: (style) The scope of the variable 'ilength' can be reduced.
[q_shared.c:1200]: (style) The scope of the variable 'c2' can be reduced.
[q_shared.c:1267]: (style) The scope of the variable 'o' can be reduced.
[q_shared.c:1305]: (style) The scope of the variable 'start' can be reduced.
[q_shared.c:1308]: (style) The scope of the variable 'o' can be reduced.
[q_shared.c:1373]: (style) The scope of the variable 'c' can be reduced.
[q_shared.h:197] -> [q_shared.c:291]: (warning) Function 'LerpAngle' argument order different: declaration 'a1, a2, frac' definition 'a2, a1, frac'
[q_shared.c:313] -> [q_shared.c:47]: (style) Local variable 'i' shadows outer variable
[q_shared.c:313] -> [q_shared.c:163]: (style) Local variable 'i' shadows outer variable
[q_shared.c:313] -> [q_shared.c:320]: (style) Local variable 'i' shadows outer variable
[q_shared.c:314] -> [q_shared.c:323]: (style) Local variable 'corners' shadows outer variable
[q_shared.c:313] -> [q_shared.c:666]: (style) Local variable 'i' shadows outer variable
[q_shared.c:313] -> [q_shared.c:772]: (style) Local variable 'i' shadows outer variable
[q_shared.c:313] -> [q_shared.c:849]: (style) Local variable 'i' shadows outer variable
[q_shared.c:313] -> [q_shared.c:1171]: (style) Local variable 'i' shadows outer variable
[u_entmgr.c:65]: (style) The scope of the variable 'spspot' can be reduced.
[x_fire.c:1182] -> [x_fire.c:1184]: (style) Variable 'fireball->s.effects' is reassigned a value before the old one has been used.
[x_fire.c:84]: (style) The scope of the variable 'v' can be reduced.
[x_fire.c:86]: (style) The scope of the variable 'eta' can be reduced.
[x_fire.c:325]: (style) Variable 'chance' is assigned a value that is never used.
[ai\ai_class_dmbot.c:1126] -> [ai\ai_class_dmbot.c:1264]: (warning) Either the condition 'weapon!=9' is redundant or the array 'AIWeapons[9]' is accessed at index 9, which is out of bounds.
[ai\ai_class_dmbot.c:1126] -> [ai\ai_class_dmbot.c:1270]: (warning) Either the condition 'weapon!=9' is redundant or the array 'AIWeapons[9]' is accessed at index 9, which is out of bounds.
[ai\ai_class_dmbot.c:1126] -> [ai\ai_class_dmbot.c:1275]: (warning) Either the condition 'weapon!=9' is redundant or the array 'AIWeapons[9]' is accessed at index 9, which is out of bounds.
[ai\ai_class_dmbot.c:438]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[ai\ai_class_dmbot.c:451]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[ai\ai_class_dmbot.c:984]: (style) Redundant condition: If 'EXPR == 8', the comparison 'EXPR' is always true.
[ai\ai_class_dmbot.c:1905] -> [ai\ai_class_dmbot.c:1906]: (style) Variable 'self->client->ping' is reassigned a value before the old one has been used.
[ai\ai_class_dmbot.c:338]: (style) The scope of the variable 'attackvector' can be reduced.
[ai\ai_class_dmbot.c:339]: (style) The scope of the variable 'dist' can be reduced.
[ai\ai_class_dmbot.c:425]: (style) The scope of the variable 'camp_targ_save' can be reduced.
[ai\ai_class_dmbot.c:427]: (style) The scope of the variable 'ent_orig' can be reduced.
[ai\ai_class_dmbot.c:430]: (style) The scope of the variable 'it' can be reduced.
[ai\ai_class_dmbot.c:431]: (style) The scope of the variable 'index' can be reduced.
[ai\ai_class_dmbot.c:683]: (style) The scope of the variable 'ammo_index' can be reduced.
[ai\ai_class_dmbot.c:684]: (style) The scope of the variable 'ammo_item' can be reduced.
[ai\ai_class_dmbot.c:937]: (style) The scope of the variable 'vdist' can be reduced.
[ai\ai_class_dmbot.c:47]: (style) Variable 'current_node_flags' is assigned a value that is never used.
[ai\ai_class_dmbot.c:135]: (style) Variable 'trace' is assigned a value that is never used.
[ai\ai_class_dmbot.c:719]: (style) Variable 'best_weight' is assigned a value that is never used.
[ai\ai_class_dmbot.c:770]: (style) Variable 'weapon_range' is assigned a value that is never used.
[ai\ai_class_dmbot.c:773]: (style) Variable 'weapon_range' is assigned a value that is never used.
[ai\ai_class_dmbot.c:776]: (style) Variable 'weapon_range' is assigned a value that is never used.
[ai\ai_class_dmbot.c:779]: (style) Variable 'weapon_range' is assigned a value that is never used.
[ai\ai_class_monster_default.c:49]: (style) Variable 'current_node_flags' is assigned a value that is never used.
[ai\ai_class_monster_default.c:50]: (style) Variable 'next_node_flags' is assigned a value that is never used.
[ai\ai_dropnodes.c:353] -> [ai\ai_dropnodes.c:357]: (style) Expression is always true because 'else if' condition is opposite to previous condition at line 353.
[ai\ai_links.c:178] -> [ai\ai_links.c:184]: (warning) Either the condition 'pLinks[n1].numLinks>16' is redundant or the array 'pLinks[n1].nodes[16]' is accessed at index 16, which is out of bounds.
[ai\ai_links.c:462]: (style) The scope of the variable 'move' can be reduced.
[ai\ai_links.c:608]: (style) The scope of the variable 'move' can be reduced.
[ai\ai_links.c:805]: (style) The scope of the variable 'link' can be reduced.
[ai\ai_links.c:958]: (style) The scope of the variable 'n2' can be reduced.
[ai\ai_links.c:1004]: (style) The scope of the variable 'n2' can be reduced.
[ai\ai_links.c:875] -> [ai\ai_links.c:914]: (style) Local variable 'link' shadows outer variable
[ai\ai_links.c:609]: (style) Variable 'movemask' is assigned a value that is never used.
[ai\ai_links.c:639]: (style) Variable 'movemask' is assigned a value that is never used.
[ai\ai_links.c:773]: (style) Variable 'xzdist' is assigned a value that is never used.
[ai\ai_main.c:34]: (style) The scope of the variable 'team' can be reduced.
[ai\ai_main.c:267]: (style) The scope of the variable 'best_broam' can be reduced.
[ai\ai_main.c:391]: (style) The scope of the variable 'nearest' can be reduced.
[ai\ai_main.c:392]: (style) The scope of the variable 'temp_distance' can be reduced.
[ai\ai_main.c:393]: (style) The scope of the variable 'distb' can be reduced.
[ai\ai_main.c:395]: (style) The scope of the variable 'j' can be reduced.
[ai\ai_main.c:395]: (style) The scope of the variable 'randseed' can be reduced.
[ai\ai_main.c:395]: (style) The scope of the variable 'k' can be reduced.
[ai\ai_main.c:401]: (style) The scope of the variable 'obj_camp' can be reduced.
[ai\ai_main.c:403]: (style) The scope of the variable 'obj_count' can be reduced.
[ai\ai_main.c:404]: (style) The scope of the variable 'rand_obj_num' can be reduced.
[ai\ai_main.c:678]: (style) The scope of the variable 'dist' can be reduced.
[ai\ai_main.c:678]: (style) The scope of the variable 'distv' can be reduced.
[ai\ai_main.c:1027]: (style) The scope of the variable 'soundfile' can be reduced.
[ai\ai_main.c:1031]: (style) The scope of the variable 'nearlength' can be reduced.
[ai\ai_main.c:1032]: (style) The scope of the variable 'checkdist' can be reduced.
[ai\ai_main.c:1033]: (style) The scope of the variable 'nearest' can be reduced.
[ai\ai_main.c:1035]: (style) The scope of the variable 'tdist' can be reduced.
[ai\ai_main.c:1036]: (style) The scope of the variable 'closest' can be reduced.
[ai\ai_main.c:1038]: (style) The scope of the variable 'dist' can be reduced.
[ai\ai_main.c:1790]: (style) The scope of the variable 'enemyview' can be reduced.
[ai\ai_main.c:1027] -> [ai\ai_main.c:1593]: (style) Local variable 'soundfile' shadows outer variable
[ai\ai_main.c:1038] -> [ai\ai_main.c:1750]: (style) Local variable 'dist' shadows outer variable
[ai\ai_navigation.c:333]: (error) Array 'check[3]' accessed at index 3, which is out of bounds.
[ai\ai_navigation.c:336] -> [ai\ai_navigation.c:363]: (warning) Either the condition 'c<4' is redundant or the array 'check[3]' is accessed at index 3, which is out of bounds.
[ai\ai_navigation.c:235]: (style) The scope of the variable 'dist' can be reduced.
[ai\ai_navigation.c:267]: (style) The scope of the variable 'dist' can be reduced.
[ai\ai_navigation.c:320]: (style) The scope of the variable 'closest' can be reduced.
[ai\ai_navigation.c:322]: (style) The scope of the variable 'node' can be reduced.
[g_local.h:1034] -> [ai\ai_navigation.c:274]: (style) Local variable 'range' shadows outer function
[ai\ai_navigation.c:320]: (style) Variable 'closest' is assigned a value that is never used.
[ai\ai_navigation.c:322]: (style) Variable 'node' is assigned a value that is never used.
[ai\ai_tools.c:224]: (style) The scope of the variable 'dist' can be reduced.
[ai\ai_tools.c:290]: (style) The scope of the variable 'dist' can be reduced.
[ai\ai_tools.c:448]: (style) The scope of the variable 'thrower' can be reduced.
[ai\ai_nodes.c:776]: (style) The scope of the variable 'heightdiff' can be reduced.
[ai\ai_nodes.c:817]: (style) The scope of the variable 'heightdiff' can be reduced.
[ai\ai_nodes.c:904]: (style) The scope of the variable 'n2' can be reduced.
[ai\ai_weapons.c:127]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[ai\ai_weapons.c:128]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[ai\ai_weapons.c:129]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[ai\ai_weapons.c:130]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[ai\ai_weapons.c:131]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[ai\ai_weapons.c:132]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[ai\ai_weapons.c:133]: (error) Array 'AIWeapons[9]' accessed at index 9, which is out of bounds.
[ai\bot_common.c:207]: (style) The scope of the variable 'cl_ent' can be reduced.
[ai\bot_common.c:276]: (style) The scope of the variable 'cl_ent' can be reduced.
[ai\bot_common.c:240]: (style) Variable 'len' is assigned a value that is never used.
[ai\bot_common.c:260]: (style) Variable 'len' is assigned a value that is never used.
[ai\bot_spawn.c:527] -> [ai\bot_spawn.c:527]: (warning) Either the condition '!bot->client' is redundant or there is possible null pointer dereference: bot->client.
[ai\bot_spawn.c:39]: (style) The scope of the variable 'cl_ent' can be reduced.
[ai\bot_spawn.c:306]: (style) The scope of the variable 'ddaynames' can be reduced.
[ai\bot_spawn.c:485]: (style) The scope of the variable 'e' can be reduced.
[ai\bot_spawn.c:485]: (style) The scope of the variable 'bot' can be reduced.
[ai\bot_spawn.c:487]: (style) The scope of the variable 'teamcount' can be reduced.
[ai\bot_spawn.c:813]: (style) The scope of the variable 'bot' can be reduced.
[ai\bot_spawn.c:571]: (style) Variable 'randnum' is assigned a value that is never used.
[ita\ita_weapon.c:335]: (style) The scope of the variable 'up' can be reduced.
[usm\usm_weapon.c:147]: (style) The scope of the variable 'ammo_item' can be reduced.
[usm\usm_weapon.c:153]: (style) Variable 'ammo_item' is assigned a value that is never used.

[MrG-DRGN]

**The (style) warnings can mostly be ignored. Here's the current state of the bugfixes branch, sans most (style) warnings
SOME of these will be addressed tonight
**

[g_ai.c:545] -> [g_ai.c:548]: (style) Condition '!self->enemy->client' is always true
[g_combat.c:290]: (style) Assignment of function parameter has no effect outside the function.
[g_ents.c:421]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:437]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:453]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:469]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_func.c:417] -> [g_func.c:415]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_func.c:1128] -> [g_func.c:1126]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_func.c:1520] -> [g_func.c:1518]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_func.c:2028] -> [g_func.c:2026]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_items.c:1611] -> [g_items.c:1610]: (warning) Either the condition 'it' is redundant or there is pointer arithmetic with NULL pointer.
[g_main.c:390]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:424]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:637]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:674]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:772]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:685] -> [g_main.c:708] -> [g_main.c:711] -> [g_main.c:329]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[g_main.c:685] -> [g_main.c:708] -> [g_main.c:709]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: check.
[g_main.c:709]: (error) Null pointer dereference
[g_maps.c:114]: (warning) fscanf() without field width limits can crash with huge input data.
[g_maps.c:131]: (warning) fscanf() without field width limits can crash with huge input data.
[g_phys.c:698] -> [g_phys.c:703]: (style) The if condition is the same as the previous if condition
[g_spawn.c:905]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_spawn.c:514]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:515]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:516]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:519]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:520]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:521]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:529]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:533]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:534]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:535]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_svcmds.c:408]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_svcmds.c:200] -> [g_svcmds.c:209]: (style) Condition 'checkexit>-1' is always true
[g_svcmds.c:217] -> [g_svcmds.c:226]: (style) Condition 'checkexit>-1' is always true
[g_svcmds.c:234] -> [g_svcmds.c:243]: (style) Condition 'checkexit>-1' is always true
[g_svcmds.c:68] -> [g_svcmds.c:70]: (style) Variable 'ent->client->resp.mos' is reassigned a value before the old one has been used.
[g_turret.c:176]: (warning) Comparison of modulo result is predetermined, because it is always less than 2.
[g_turret.c:98] -> [g_turret.c:96]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[g_turretx.c:106] -> [g_turretx.c:103]: (warning) Either the condition 'if(other)' is redundant or there is possible null pointer dereference: other.
[m_civilian.c:150] -> [m_civilian.c:160]: (style) Variable 'tr' is reassigned a value before the old one has been used.
[p_client.c:2765]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:3151]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:4299]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:4533]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:1874] -> [p_client.c:1883]: (style) Condition '!spot' is always true
[p_client.c:1925] -> [p_client.c:1945]: (style) Condition '!spot' is always true
[p_client.c:4495]: (error) Memory leak: f
[p_client.c:715] -> [p_client.c:707]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[p_client.c:715] -> [p_client.c:710]: (warning) Either the condition 'self->client&&!self->ai&&attacker' is redundant or there is possible null pointer dereference: attacker.
[p_client.c:2636] -> [p_client.c:2730]: (style) Variable 's' is reassigned a value before the old one has been used.
[g_local.h:985] -> [p_client.c:1542]: (style) Local variable 'range' shadows outer function
[p_hud.c:1338]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1341]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1345]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1348]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1350]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1357]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1361]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_hud.c:1386]: (error) Undefined behavior: Variable 'string' is used as parameter and destination in sprintf().
[p_observer.c:114] -> [p_observer.c:132]: (style) Condition 'ent->owner->velocity[2]>0' is always false
[p_view.c:2573] -> [p_view.c:2578]: (style) Condition 'ent->client->layout_type==SHOW_PSCORES' is always false
[p_view.c:2972] -> [p_view.c:2975]: (warning) Either the condition 'ent->client->last_pos1' is redundant or there is possible null pointer dereference: ent->client->last_pos1.
[p_weapon.c:186] -> [p_weapon.c:259]: (warning) Either the condition 'other->client' is redundant or there is possible null pointer dereference: other->client.
[p_weapon.c:1107] -> [p_weapon.c:1130]: (warning) Either the condition 'ent->client' is redundant or there is possible null pointer dereference: ent->client.
[p_weapon.c:1719] -> [p_weapon.c:1722]: (warning) Either the condition 'ent->client' is redundant or there is possible null pointer dereference: ent->client.
[q_shared.c:1270] -> [q_shared.c:1272]: (warning) Opposite inner 'if' condition leads to a dead code block.
[q_shared.c:1316] -> [q_shared.c:1318]: (warning) Opposite inner 'if' condition leads to a dead code block.
[q_shared.c:1382] -> [q_shared.c:1358]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[q_shared.c:1382] -> [q_shared.c:1370]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[q_shared.c:1382] -> [q_shared.c:1376]: (warning) Either the condition '!value' is redundant or there is possible null pointer dereference: value.
[q_shared.c:258]: (portability) Casting from float * to signed int * is not portable due to different binary data representations on different platforms.
[q_shared.c:260]: (portability) Casting between signed int * and float * which have an incompatible binary data representation.
[q_shared.h:220] -> [q_shared.c:283]: (warning) Function 'LerpAngle' argument order different: declaration 'a1, a2, frac' definition 'a2, a1, frac'
[ai\ai_movement.c:366]: (style) Modulo of one is always equal to zero

[InFerYes]

Thanks for working it down to a smaller list.

[MrG-DRGN]

Still working on it...

[g_ents.c:427]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:443]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:459]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_ents.c:475]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_func.c:417] -> [g_func.c:415]: (warning) Either the condition 'other' is redundant or there is possible null pointer dereference: other.
[g_func.c:1128] -> [g_func.c:1126]: (warning) Either the condition 'other' is redundant or there is possible null pointer dereference: other.
[g_func.c:1520] -> [g_func.c:1518]: (warning) Either the condition 'other' is redundant or there is possible null pointer dereference: other.
[g_func.c:2028] -> [g_func.c:2026]: (warning) Either the condition 'other' is redundant or there is possible null pointer dereference: other.
[g_items.c:1611] -> [g_items.c:1610]: (warning) Either the condition 'it' is redundant or there is pointer arithmetic with NULL pointer.
[g_main.c:394]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:428]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:641]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:678]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:776]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_main.c:689] -> [g_main.c:712] -> [g_main.c:715] -> [g_main.c:333]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: map.
[g_main.c:689] -> [g_main.c:712] -> [g_main.c:713]: (warning) Either the condition 't==((void*)0)' is redundant or there is possible null pointer dereference: check.
[g_main.c:713]: (error) Null pointer dereference
[g_spawn.c:899]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_spawn.c:515]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:516]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:517]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:523]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:527]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:528]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_spawn.c:529]: (portability) Casting between unsigned char * and float * which have an incompatible binary data representation.
[g_svcmds.c:408]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[g_svcmds.c:200] -> [g_svcmds.c:209]: (style) Condition 'checkexit>-1' is always true
[g_svcmds.c:217] -> [g_svcmds.c:226]: (style) Condition 'checkexit>-1' is always true
[g_svcmds.c:234] -> [g_svcmds.c:243]: (style) Condition 'checkexit>-1' is always true
[g_turret.c:98] -> [g_turret.c:96]: (warning) Either the condition 'other' is redundant or there is possible null pointer dereference: other.
[g_combat.c:167] -> [g_combat.c:173]: (style) Condition 'targ->client' is always true
[g_combat.c:214] -> [g_combat.c:216]: (style) Condition 'attacker->client->resp.team_on==targ->client->resp.team_on' is always true
[g_combat.c:214] -> [g_combat.c:221]: (style) Condition 'attacker->client->resp.team_on!=targ->client->resp.team_on' is always false
[g_combat.c:210] -> [g_combat.c:231]: (style) Condition 'targ->client' is always true
[p_observer.c:114] -> [p_observer.c:132]: (style) Condition 'ent->owner->velocity[2]>0' is always false
[g_weapon.c:1312] -> [g_weapon.c:1281]: (warning) Either the condition 'ent->owner->client' is redundant or there is possible null pointer dereference: ent->owner->client.
[g_weapon.c:2266] -> [g_weapon.c:2239]: (warning) Either the condition 'self->client' is redundant or there is possible null pointer dereference: self->client.
[q_shared.c:1364] -> [q_shared.c:1366]: (warning) Opposite inner 'if' condition leads to a dead code block.
[q_shared.c:1410] -> [q_shared.c:1412]: (warning) Opposite inner 'if' condition leads to a dead code block.
[q_shared.c:1264] -> [q_shared.c:1267]: (style) Condition 'decSize>0' is always true
[q_shared.c:258]: (portability) Casting from float * to signed int * is not portable due to different binary data representations on different platforms.
[q_shared.c:260]: (portability) Casting between signed int * and float * which have an incompatible binary data representation.
[p_view.c:2573] -> [p_view.c:2578]: (style) Condition 'ent->client->layout_type==SHOW_PSCORES' is always false
[p_view.c:2972] -> [p_view.c:2975]: (warning) Either the condition 'ent->client->last_pos1' is redundant or there is possible null pointer dereference: ent->client->last_pos1.
[ai\ai_main.c:1088] -> [ai\ai_main.c:1065]: (warning) Either the condition 'e->client' is redundant or there is possible null pointer dereference: e->client.
[p_client.c:2775]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:3161]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:4314]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:4554]: (style) Obsolete function 'strdup' called. It is recommended to use '_strdup' instead.
[p_client.c:1882] -> [p_client.c:1891]: (style) Condition '!spot' is always true
[p_client.c:1933] -> [p_client.c:1953]: (style) Condition '!spot' is always true
[p_client.c:2632] -> [p_client.c:2651]: (style) Condition 'ent' is always true
[p_client.c:2651] -> [p_client.c:2737]: (warning) Either the condition 'ent' is redundant or there is possible null pointer dereference: ent.