wesleyjohnson
Member
Posts: 486
Registered: 04-09 |
Just did bobbing code changes in DoomLegacy (no, I am already full time tied up by DoomLegacy). That code is all over the place.
That is because it is not independent of player movement.
Every attempt to fix bobbing resulted in compromises that would affect player movement, so Killough finally came up with the independent player bobbing momentum fields, player->momx and player->momy (which in DoomLegacy are bob_momx, and bob_momy). Anything derived from sources earlier than MBF do not have those fields (only have player->mo->momx, player->mo->momy).
First thing, make a copy of your working directory, and only work in your experimental directory. When you are satisfied with your mods, then do a DIFF against the original that you started with, to create a patch file. Then read that patch file carefully, so you can remove patches that are not part of your fix.
To test an idea use #if 0 ... #endif around that parts that you are cutting out.
Mark your code with comments like
#if 1
// [WDJ] Test of new bobbing code 6/24/2011
..
#else
// [WDJ] Original bobbing code
..
#endif
Grep will find all the uses of most variable uses. It would be better if the authors had not used the same field names in so many structures.
Many times end up using multiple greps.
grep "bob" * | grep "momentum" | less
It would be useful to DIFF against a close relative that has the bobbing the way you want it. But Chocolate Doom may emulate the original behavior but I expect the internal coding is all changed.
That means you have to do the comparisons by eye.
All bobbing is closely tied to changes in player->mo->momx, player->mo->momy.
The orignal updated this in some closely related functions, P_PlayerMove, P_Bob, P_Thrust.
You will see player->momx and player->bob both in one function.
Only change the player->bob (like changing the sign of what is added to it). Changing player->mo->momx and player->mo->momy will affect player movement, with many consequences.
Not having Chocolate Doom source, I can only guess whether it keeps the original names, or has new functions, and new structure.
DoomLegacy has changed many of its function names too (to make grep more useful if nothing else).
Beware of adding anything to info.c structures, they are indexed by other definitions made in other files. Only add to the end of a structure, and then you will have to also create definition names to index the new entries. See how any advanced port added Heretic to the original Doom info.c entries.
I do not remember anything about "0.4 alpha bobbing", do not know what it is.
Last edited by wesleyjohnson on 06-27-11 at 22:21
|
06-27-11 20:34 
