From 9828b0228e1f82b3097e100ad9e1363abc5b76dd Mon Sep 17 00:00:00 2001 From: Christoph Oelckers Date: Sat, 17 Apr 2021 10:22:00 +0200 Subject: [PATCH] - made type of shade variables consistent. Some were char, some signed char and the initializers needed type casts. This wasn't serious but created a lot of casting noise. --- source/games/blood/src/actor.cpp | 268 +++++++++++++++---------------- source/games/blood/src/actor.h | 16 +- 2 files changed, 142 insertions(+), 142 deletions(-) diff --git a/source/games/blood/src/actor.cpp b/source/games/blood/src/actor.cpp index e892cc8db..28bf6f1fe 100644 --- a/source/games/blood/src/actor.cpp +++ b/source/games/blood/src/actor.cpp @@ -662,7 +662,7 @@ const ITEMDATA gItemData[] = { { 0, 2552, - (char)-8, + -8, 0, 32, 32, @@ -671,7 +671,7 @@ const ITEMDATA gItemData[] = { { 0, 2553, - (char)-8, + -8, 0, 32, 32, @@ -680,7 +680,7 @@ const ITEMDATA gItemData[] = { { 0, 2554, - (char)-8, + -8, 0, 32, 32, @@ -689,7 +689,7 @@ const ITEMDATA gItemData[] = { { 0, 2555, - (char)-8, + -8, 0, 32, 32, @@ -698,7 +698,7 @@ const ITEMDATA gItemData[] = { { 0, 2556, - (char)-8, + -8, 0, 32, 32, @@ -707,7 +707,7 @@ const ITEMDATA gItemData[] = { { 0, 2557, - (char)-8, + -8, 0, 32, 32, @@ -716,7 +716,7 @@ const ITEMDATA gItemData[] = { { 0, -1, - (char)-8, + -8, 0, 255, 255, @@ -725,7 +725,7 @@ const ITEMDATA gItemData[] = { { 0, 519, - (char)-8, + -8, 0, 48, 48, @@ -734,7 +734,7 @@ const ITEMDATA gItemData[] = { { 0, 822, - (char)-8, + -8, 0, 40, 40, @@ -743,7 +743,7 @@ const ITEMDATA gItemData[] = { { 0, 2169, - (char)-8, + -8, 0, 40, 40, @@ -752,7 +752,7 @@ const ITEMDATA gItemData[] = { { 0, 2433, - (char)-8, + -8, 0, 40, 40, @@ -761,7 +761,7 @@ const ITEMDATA gItemData[] = { { 0, 517, - (char)-8, + -8, 0, 40, 40, @@ -770,7 +770,7 @@ const ITEMDATA gItemData[] = { { 0, 783, - (char)-8, + -8, 0, 40, 40, @@ -779,7 +779,7 @@ const ITEMDATA gItemData[] = { { 0, 896, - (char)-8, + -8, 0, 40, 40, @@ -788,7 +788,7 @@ const ITEMDATA gItemData[] = { { 0, 825, - (char)-8, + -8, 0, 40, 40, @@ -797,7 +797,7 @@ const ITEMDATA gItemData[] = { { 0, 827, - (char)-8, + -8, 0, 40, 40, @@ -806,7 +806,7 @@ const ITEMDATA gItemData[] = { { 0, 828, - (char)-8, + -8, 0, 40, 40, @@ -815,7 +815,7 @@ const ITEMDATA gItemData[] = { { 0, 829, - (char)-8, + -8, 0, 40, 40, @@ -824,7 +824,7 @@ const ITEMDATA gItemData[] = { { 0, 830, - (char)-8, + -8, 0, 80, 64, @@ -833,7 +833,7 @@ const ITEMDATA gItemData[] = { { 0, 831, - (char)-8, + -8, 0, 40, 40, @@ -842,7 +842,7 @@ const ITEMDATA gItemData[] = { { 0, 863, - (char)-8, + -8, 0, 40, 40, @@ -851,7 +851,7 @@ const ITEMDATA gItemData[] = { { 0, 760, - (char)-8, + -8, 0, 40, 40, @@ -860,7 +860,7 @@ const ITEMDATA gItemData[] = { { 0, 836, - (char)-8, + -8, 0, 40, 40, @@ -869,7 +869,7 @@ const ITEMDATA gItemData[] = { { 0, 851, - (char)-8, + -8, 0, 40, 40, @@ -878,7 +878,7 @@ const ITEMDATA gItemData[] = { { 0, 2428, - (char)-8, + -8, 0, 40, 40, @@ -887,7 +887,7 @@ const ITEMDATA gItemData[] = { { 0, 839, - (char)-8, + -8, 0, 40, 40, @@ -896,7 +896,7 @@ const ITEMDATA gItemData[] = { { 0, 768, - (char)-8, + -8, 0, 64, 64, @@ -905,7 +905,7 @@ const ITEMDATA gItemData[] = { { 0, 840, - (char)-8, + -8, 0, 48, 48, @@ -914,7 +914,7 @@ const ITEMDATA gItemData[] = { { 0, 841, - (char)-8, + -8, 0, 48, 48, @@ -923,7 +923,7 @@ const ITEMDATA gItemData[] = { { 0, 842, - (char)-8, + -8, 0, 48, 48, @@ -932,7 +932,7 @@ const ITEMDATA gItemData[] = { { 0, 843, - (char)-8, + -8, 0, 48, 48, @@ -941,7 +941,7 @@ const ITEMDATA gItemData[] = { { 0, 683, - (char)-8, + -8, 0, 40, 40, @@ -950,7 +950,7 @@ const ITEMDATA gItemData[] = { { 0, 521, - (char)-8, + -8, 0, 40, 40, @@ -959,7 +959,7 @@ const ITEMDATA gItemData[] = { { 0, 604, - (char)-8, + -8, 0, 40, 40, @@ -968,7 +968,7 @@ const ITEMDATA gItemData[] = { { 0, 520, - (char)-8, + -8, 0, 40, 40, @@ -977,7 +977,7 @@ const ITEMDATA gItemData[] = { { 0, 803, - (char)-8, + -8, 0, 40, 40, @@ -986,7 +986,7 @@ const ITEMDATA gItemData[] = { { 0, 518, - (char)-8, + -8, 0, 40, 40, @@ -995,7 +995,7 @@ const ITEMDATA gItemData[] = { { 0, 522, - (char)-8, + -8, 0, 40, 40, @@ -1004,7 +1004,7 @@ const ITEMDATA gItemData[] = { { 0, 523, - (char)-8, + -8, 0, 40, 40, @@ -1013,7 +1013,7 @@ const ITEMDATA gItemData[] = { { 0, 837, - (char)-8, + -8, 0, 80, 64, @@ -1022,7 +1022,7 @@ const ITEMDATA gItemData[] = { { 0, 2628, - (char)-8, + -8, 0, 64, 64, @@ -1031,7 +1031,7 @@ const ITEMDATA gItemData[] = { { 0, 2586, - (char)-8, + -8, 0, 64, 64, @@ -1040,7 +1040,7 @@ const ITEMDATA gItemData[] = { { 0, 2578, - (char)-8, + -8, 0, 64, 64, @@ -1049,7 +1049,7 @@ const ITEMDATA gItemData[] = { { 0, 2602, - (char)-8, + -8, 0, 64, 64, @@ -1058,7 +1058,7 @@ const ITEMDATA gItemData[] = { { 0, 2594, - (char)-8, + -8, 0, 64, 64, @@ -1067,7 +1067,7 @@ const ITEMDATA gItemData[] = { { 0, 753, - (char)-8, + -8, 0, 64, 64, @@ -1076,7 +1076,7 @@ const ITEMDATA gItemData[] = { { 0, 753, - (char)-8, + -8, 7, 64, 64, @@ -1085,7 +1085,7 @@ const ITEMDATA gItemData[] = { { 0, 3558, - (char)-128, + -128, 0, 64, 64, @@ -1094,7 +1094,7 @@ const ITEMDATA gItemData[] = { { 0, 3558, - (char)-128, + -128, 7, 64, 64, @@ -1106,7 +1106,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 618, - (char)-8, + -8, 0, 40, 40, @@ -1117,7 +1117,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 589, - (char)-8, + -8, 0, 48, 48, @@ -1128,7 +1128,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 589, - (char)-8, + -8, 0, 48, 48, @@ -1139,7 +1139,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 809, - (char)-8, + -8, 0, 48, 48, @@ -1150,7 +1150,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 811, - (char)-8, + -8, 0, 48, 48, @@ -1161,7 +1161,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 810, - (char)-8, + -8, 0, 48, 48, @@ -1172,7 +1172,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 820, - (char)-8, + -8, 0, 24, 24, @@ -1183,7 +1183,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 619, - (char)-8, + -8, 0, 48, 48, @@ -1194,7 +1194,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 812, - (char)-8, + -8, 0, 48, 48, @@ -1205,7 +1205,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 813, - (char)-8, + -8, 0, 48, 48, @@ -1216,7 +1216,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 525, - (char)-8, + -8, 0, 48, 48, @@ -1227,7 +1227,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 814, - (char)-8, + -8, 0, 48, 48, @@ -1238,7 +1238,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 817, - (char)-8, + -8, 0, 48, 48, @@ -1249,7 +1249,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 548, - (char)-8, + -8, 0, 24, 24, @@ -1260,7 +1260,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 0, - (char)-8, + -8, 0, 48, 48, @@ -1271,7 +1271,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 0, - (char)-8, + -8, 0, 48, 48, @@ -1282,7 +1282,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 816, - (char)-8, + -8, 0, 48, 48, @@ -1293,7 +1293,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 818, - (char)-8, + -8, 0, 48, 48, @@ -1304,7 +1304,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 819, - (char)-8, + -8, 0, 48, 48, @@ -1315,7 +1315,7 @@ const AMMOITEMDATA gAmmoItemData[] = { { 0, 801, - (char)-8, + -8, 0, 48, 48, @@ -1340,7 +1340,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1351,7 +1351,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 559, - (char)-8, + -8, 0, 48, 48, @@ -1362,7 +1362,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 558, - (char)-8, + -8, 0, 48, 48, @@ -1373,7 +1373,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 524, - (char)-8, + -8, 0, 48, 48, @@ -1384,7 +1384,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 525, - (char)-8, + -8, 0, 48, 48, @@ -1395,7 +1395,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 539, - (char)-8, + -8, 0, 48, 48, @@ -1406,7 +1406,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 526, - (char)-8, + -8, 0, 48, 48, @@ -1417,7 +1417,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1428,7 +1428,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 618, - (char)-8, + -8, 0, 48, 48, @@ -1439,7 +1439,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 589, - (char)-8, + -8, 0, 48, 48, @@ -1450,7 +1450,7 @@ const WEAPONITEMDATA gWeaponItemData[] = { { 0, 800, - (char)-8, + -8, 0, 48, 48, @@ -1468,7 +1468,7 @@ const MissileType missileInfo[] = { 512, 40, 40, - (char)-16, + -16, 16, }, // Regular flare @@ -1478,7 +1478,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 32, }, // Tesla alt @@ -1488,7 +1488,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 32, }, // Flare alt @@ -1498,7 +1498,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 4, }, // Spray flame @@ -1508,7 +1508,7 @@ const MissileType missileInfo[] = { 0, 24, 24, - (char)-128, + -128, 16, }, // Fireball @@ -1518,7 +1518,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 32, }, // Tesla regular @@ -1528,7 +1528,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 16, }, // EctoSkull @@ -1538,7 +1538,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-24, + -24, 32, }, // Hellhound flame @@ -1548,7 +1548,7 @@ const MissileType missileInfo[] = { 0, 24, 24, - (char)-128, + -128, 16, }, // Puke @@ -1558,7 +1558,7 @@ const MissileType missileInfo[] = { 0, 16, 16, - (char)-16, + -16, 16, }, // Reserved @@ -1568,7 +1568,7 @@ const MissileType missileInfo[] = { 0, 8, 8, - (char)0, + 0, 16, }, // Stone gargoyle projectile @@ -1578,7 +1578,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 16, }, // Napalm launcher @@ -1588,7 +1588,7 @@ const MissileType missileInfo[] = { 0, 30, 30, - (char)-128, + -128, 24, }, // Cerberus fireball @@ -1598,7 +1598,7 @@ const MissileType missileInfo[] = { 0, 30, 30, - (char)-128, + -128, 24, }, // Tchernobog fireball @@ -1608,7 +1608,7 @@ const MissileType missileInfo[] = { 0, 24, 24, - (char)-128, + -128, 16, }, // Regular life leech @@ -1618,7 +1618,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 16, }, // Dropped life leech (enough ammo) @@ -1628,7 +1628,7 @@ const MissileType missileInfo[] = { 0, 16, 16, - (char)-128, + -128, 16, }, // Dropped life leech (no ammo) @@ -1638,7 +1638,7 @@ const MissileType missileInfo[] = { 0, 32, 32, - (char)-128, + -128, 16, } }; @@ -1654,7 +1654,7 @@ const THINGINFO thingInfo[] = { 80, 384, 907, - (char)0, + 0, 0, 0, 0, @@ -1671,7 +1671,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 3444, - (char)-16, + -16, 0, 32, 32, @@ -1687,7 +1687,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 3457, - (char)-16, + -16, 0, 32, 32, @@ -1703,7 +1703,7 @@ const THINGINFO thingInfo[] = { 80, 0, 739, - (char)0, + 0, 0, 0, 0, @@ -1719,7 +1719,7 @@ const THINGINFO thingInfo[] = { 80, 0, 642, - (char)0, + 0, 0, 0, 0, @@ -1735,7 +1735,7 @@ const THINGINFO thingInfo[] = { 0, 0, 462, - (char)0, + 0, 0, 0, 0, @@ -1751,7 +1751,7 @@ const THINGINFO thingInfo[] = { 0, 0, 266, - (char)0, + 0, 0, 0, 0, @@ -1767,7 +1767,7 @@ const THINGINFO thingInfo[] = { 0, 0, 796, - (char)0, + 0, 0, 0, 0, @@ -1783,7 +1783,7 @@ const THINGINFO thingInfo[] = { 0, 0, 1127, - (char)0, + 0, 0, 0, 0, @@ -1799,7 +1799,7 @@ const THINGINFO thingInfo[] = { 0, 0, 1142, - (char)0, + 0, 0, 0, 0, @@ -1815,7 +1815,7 @@ const THINGINFO thingInfo[] = { 0, 0, 1069, - (char)0, + 0, 0, 0, 0, @@ -1831,7 +1831,7 @@ const THINGINFO thingInfo[] = { 0, 0, 483, - (char)0, + 0, 0, 0, 0, @@ -1847,7 +1847,7 @@ const THINGINFO thingInfo[] = { 0, 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1863,7 +1863,7 @@ const THINGINFO thingInfo[] = { 0, 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1879,7 +1879,7 @@ const THINGINFO thingInfo[] = { 0, 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1895,7 +1895,7 @@ const THINGINFO thingInfo[] = { 0, 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1911,7 +1911,7 @@ const THINGINFO thingInfo[] = { 0, 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1927,7 +1927,7 @@ const THINGINFO thingInfo[] = { 0, 0, -1, - (char)0, + 0, 0, 0, 0, @@ -1943,7 +1943,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 3422, - (char)-32, + -32, 0, 32, 32, @@ -1959,7 +1959,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 3433, - (char)-32, + -32, 0, 32, 32, @@ -1975,7 +1975,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 3467, - (char)-128, + -128, 0, 32, 32, @@ -1991,7 +1991,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 1462, - (char)0, + 0, 0, 32, 32, @@ -2007,7 +2007,7 @@ const THINGINFO thingInfo[] = { 1600, 256, -1, - (char)0, + 0, 0, 0, 0, @@ -2023,7 +2023,7 @@ const THINGINFO thingInfo[] = { 0, 0, 1147, - (char)0, + 0, 10, 0, 0, @@ -2039,7 +2039,7 @@ const THINGINFO thingInfo[] = { 0, 0, 1160, - (char)0, + 0, 2, 0, 0, @@ -2055,7 +2055,7 @@ const THINGINFO thingInfo[] = { 0, 257, -1, - (char)0, + 0, 0, 0, 0, @@ -2071,7 +2071,7 @@ const THINGINFO thingInfo[] = { 0, 257, -1, - (char)0, + 0, 0, 0, 0, @@ -2087,7 +2087,7 @@ const THINGINFO thingInfo[] = { 1280, 257, 3405, - (char)0, + 0, 0, 40, 40, @@ -2103,7 +2103,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 3281, - (char)-128, + -128, 0, 32, 32, @@ -2119,7 +2119,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 2020, - (char)-128, + -128, 0, 32, 32, @@ -2135,7 +2135,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 1860, - (char)-128, + -128, 0, 32, 32, @@ -2151,7 +2151,7 @@ const THINGINFO thingInfo[] = { 1600, 257, 800, - (char)-128, + -128, 0, 48, 48, @@ -2167,7 +2167,7 @@ const THINGINFO thingInfo[] = { 1600, 0, 2443, - (char)-128, + -128, 0, 16, 16, @@ -2183,7 +2183,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 3444, - (char)-16, + -16, 7, 32, 32, @@ -2199,7 +2199,7 @@ const THINGINFO thingInfo[] = { 1600, 256, 1462, - (char)0, + 0, 0, 32, 32, @@ -2215,7 +2215,7 @@ const THINGINFO thingInfo[] = { 1600, 257, 800, - (char)-128, + -128, 0, 44, 44, diff --git a/source/games/blood/src/actor.h b/source/games/blood/src/actor.h index 590927a7b..28a80be2c 100644 --- a/source/games/blood/src/actor.h +++ b/source/games/blood/src/actor.h @@ -76,7 +76,7 @@ struct THINGINFO int dmgResist; // damage resistance short cstat; short picnum; - char shade; + int8_t shade; uint8_t pal; uint8_t xrepeat; // xrepeat uint8_t yrepeat; // yrepeat @@ -87,8 +87,8 @@ struct AMMOITEMDATA { short cstat; short picnum; - char shade; - char pal; + int8_t shade; + uint8_t pal; uint8_t xrepeat; uint8_t yrepeat; short count; @@ -100,8 +100,8 @@ struct WEAPONITEMDATA { short cstat; short picnum; - char shade; - char pal; + int8_t shade; + uint8_t pal; uint8_t xrepeat; uint8_t yrepeat; short type; @@ -113,8 +113,8 @@ struct ITEMDATA { short cstat; short picnum; - char shade; - char pal; + int8_t shade; + uint8_t pal; uint8_t xrepeat; uint8_t yrepeat; short packSlot; @@ -127,7 +127,7 @@ struct MissileType int angleOfs; uint8_t xrepeat; uint8_t yrepeat; - char shade; + int8_t shade; uint8_t clipDist; };