Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Desync on Strain map07 #34

Closed
chocolate-import opened this issue Dec 8, 2013 · 3 comments
Closed

Desync on Strain map07 #34

chocolate-import opened this issue Dec 8, 2013 · 3 comments

Comments

@chocolate-import
Copy link

The following bug was originally reported on Sourceforge by lemonzest, 2008-01-10 00:13:59:

Hi

desync on level exit map07 of strain.wad+strain.deh which does not happen in either doom2.exe or prboom-plus

demo is ST07-237.LMP from http://competn.doom2.net/pub/sda/p-s/stmax-av.zip

demo plays back fine when using -spechit 0x01D6CF98 (same value as PrBoom-plus???)
@chocolate-import
Copy link
Author

Comment by lemonzest, 2008-01-10 00:30:41:

Logged In: YES
user_id=1672408
Originator: YES

it appears that chocolate doom is using the old spechit magic number from prboom-plus which has since been commented out and replaced by 0x01C09C98, also the demo playes back with this number

details are

chocolates current spechit magic number : 0x84f968e8
prboom-plus current spechit magic number : 0x01C09C98
prboom-plus old spechit magic number : 0x84000000

@chocolate-import
Copy link
Author

Comment by fraggle, 2008-01-10 09:25:24:

Logged In: YES
user_id=25884
Originator: NO

I was using a different default spechit magic value because it appeared to allow more demos to work "out of the box" without a -spechit parameter being specified. However, in this case, it caused a demo that works OOB on prboom-plus to fail. For consistency with prboom-plus, I've switched to the same spechits default value that it uses; at least that way, the spechits overrun list on Doomworld is applicable to Chocolate Doom as well.

@chocolate-import
Copy link
Author

Comment by fraggle, 2008-01-10 09:25:44:

  • status: open --> closed-fixed

jmtd pushed a commit to jmtd/chocolate-doom that referenced this issue Jul 14, 2015
@fabiangreffrath
fix crashes with IDMUS
a71683f 
specifically prevent crash with IDMUS00 in Doom II and
with IDMUS0x or IDMUSx0 in Doom

fixes chocolate-doom#34

also, fix/add comments and declare "extern boolean P_CheckAmmo()"
where it is actually needed
drfrag666 pushed a commit to drfrag666/RUDE that referenced this issue Jun 4, 2018
@fabiangreffrath @drfrag666
fix crashes with IDMUS
f9afe75 
specifically prevent crash with IDMUS00 in Doom II and
with IDMUS0x or IDMUSx0 in Doom

fixes chocolate-doom#34

# Conflicts:
#	src/doom/st_stuff.c
drfrag666 pushed a commit to drfrag666/RUDE that referenced this issue Oct 19, 2019
@fabiangreffrath @drfrag666
fix crashes with IDMUS
e75940e 
specifically prevent crash with IDMUS00 in Doom II and
with IDMUS0x or IDMUSx0 in Doom

fixes chocolate-doom#34

also, fix/add comments and declare "extern boolean P_CheckAmmo()"
where it is actually needed

# Conflicts:
#	src/doom/st_stuff.c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chocolate-import