0002969: Online desync with slamming actors and the Idle state

Chat on our Discord Server Get the latest version: 3.1
Source Code

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0002969

Zandronum

[All Projects] Bug

public

2016-12-29 10:22

2018-09-30 21:41

Reporter

Edward-san

Assigned To

Edward-san

Priority

normal

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

fixed

Platform

Linux

Ubuntu

OS Version

14.04 x86-64

Product Version

2.1

Target Version

3.0

Fixed in Version

3.0

Summary

0002969: Online desync with slamming actors and the Idle state

Description

I noticed by code inspection that the server erroneously told the clients that some actors (mostly the ones doing a Slam/Skullattack) would jump to the Spawn state instead of the Idle state if such state exists.

Additional Information

Old time telltale: since when ZDoom svn r876 was ported (in skulltag svn r1308), the online code was never adjusted to use it.

One additional note: 3.0 changes also A_MinotaurChase to use the idle state, but looking at the code, there's a bigger problem that could explain possibly ticket 0002382: the server mistakenly told the clients to change the state of the Minotaur instead of the MinotaurFriend.

Attached Files

Relationships

Notes
(0016566) Edward-san (developer) 2016-12-29 10:36	Can somebody build'https://bitbucket.org/zandronum/zandronum-sandbox/commits/03f4e946938f91d8dcaf709c704d5caee686393c [^]' and check if nothing is broken?

(0016567) WaTaKiD (updater) 2016-12-29 14:41	this 3.0 build contains the above commit: 'https://www.dropbox.com/s/p8574ordh5e3efm/zandronum-3.0-r161229-1025-03f4e94-windows.zip?dl=0 [^]'

(0017075) Torr Samaho (administrator) 2017-04-02 09:35	Since there was no feedback for months and the patch looks reasonable, I added it.

**Issue Community Support**
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why.
Supporters:	No one explicitly supports this issue yet.
Opponents:	No one explicitly opposes this issue yet.

Issue History
Date Modified	Username	Field	Change
2016-12-29 10:22	Edward-san	New Issue
2016-12-29 10:22	Edward-san	Status	new => assigned
2016-12-29 10:22	Edward-san	Assigned To	=> Edward-san
2016-12-29 10:36	Edward-san	Note Added: 0016566
2016-12-29 10:36	Edward-san	Status	assigned => feedback
2016-12-29 14:41	WaTaKiD	Note Added: 0016567
2017-04-02 09:35	Torr Samaho	Note Added: 0017075
2017-04-02 09:35	Torr Samaho	Status	feedback => needs testing
2017-09-07 20:35	Dusk	Status	needs testing => resolved
2017-09-07 20:35	Dusk	Fixed in Version	=> 3.0
2017-09-07 20:35	Dusk	Resolution	open => fixed
2018-09-30 21:41	Blzut3	Status	resolved => closed

Questions or other issues? Contact Us.

Links

Doom and Doom II are the property of id Software.