SPR2_DKR<A TO K EXCLUDING F>: New SPR2s which allow character WADs to define unique kart explosion gibs
This merge request is pretty much a sequel to an earlier one now (with great thanks and appreciation) already merged into Kart Krew's master branch.
In that merge request, the kart husk that appears upon a racer's death became customizable as a part of a racer's skin graphic lumps with the new SPR2
lump DKRT
.
In this merge request, pretty much the same thing has been implemented only it is now about a dozen new SPR2
lumps (SPR2_DKR<A TO K EXCLUDING F>
) each to represent a customizable version of a different "gib" that spawns upon a kart's death explosion.
I don't have much else to say about this merge request that wasn't covered in the one linked earlier, so below are some demonstration GIFs and videos:
Merge request reports
Activity
added 9 commits
-
c2cdf1f6...271c5312 - 7 commits from branch
KartKrew:master
- ced2db6f - Fully implemented the goal of the branch in one commit. Nice!
- 08f9fdfc - Switched from a sprite-to-state mapper to just feeding states directly...
-
c2cdf1f6...271c5312 - 7 commits from branch
added Feature label
added 21 commits
-
08f9fdfc...a4d304a1 - 19 commits from branch
KartKrew:master
- 5c69bdd0 - Fully implemented the goal of the branch in one commit. Nice!
- e17fbb9a - Switched from a sprite-to-state mapper to just feeding states directly...
-
08f9fdfc...a4d304a1 - 19 commits from branch
added 14 commits
-
e17fbb9a...c192f13d - 13 commits from branch
KartKrew:master
- a8a2dd93 - SPR2_DKRA through to SPR2_DKRK (excluding SPR2_DKRF) can now optionally be...
-
e17fbb9a...c192f13d - 13 commits from branch
added 5 commits
-
a8a2dd93...e6eaaebd - 4 commits from branch
KartKrew:master
- 69431edb - SPR2_DKRA through to SPR2_DKRK (excluding SPR2_DKRF) can now optionally be...
-
a8a2dd93...e6eaaebd - 4 commits from branch
added 9 commits
-
69431edb...b9c37231 - 8 commits from branch
KartKrew:master
- 7c902b3a - SPR2_DKRA through to SPR2_DKRK (excluding SPR2_DKRF) can now optionally be...
-
69431edb...b9c37231 - 8 commits from branch
- Resolved by toaster
Pretty good. Code is a little messy in terms of its use of whitespace and the order of checks, but nothing stands out as cinemasins ding-able. Why did you skip F, though? Does
DKRF
exist in notable custom addon(s) already? Not a huge issue if so, would rather have more logical lumpnames for new addons
- Resolved by Spring Thing
- Resolved by Spring Thing
- Resolved by Spring Thing
- Resolved by Spring Thing
- Resolved by Spring Thing