MantisBT - Zandronum
View Issue Details
0002440Zandronum[All Projects] Bugpublic2015-09-06 11:442018-09-30 22:18
Zalewa 
Dusk 
normalminoralways
closedfixed 
2.1 
3.0 
0002440: Clientside DECORATE negative args[] wonkiness in A_SpawnItemEx with SXF_CLIENTSIDE
Consider following decorate code

Actor Spawner 195
{
  states
  {
    Spawn:
    Active:
      TNT1 A 0
      TNT1 A 5 A_SpawnItemEx("Splat", 0.0, 0.0, 32.0, \
                             Args[0], Args[0], Args[0], \
                             0, SXF_CLIENTSIDE)
      Loop
  }
}

actor Splat
{
  +NoGravity
  States
  {
  Spawn:
    BLUD CBA 8
    Stop
  }
}


Assigning a negative value to args[0] will set negative X/Y/Z velocity for spawned object. This works correctly offline. However, when run on server, things will go crazy. Removing SXF_CLIENTSIDE flag fixes the issue.
1. Download attached "argsnegative.wad"
2. Run MAP01 offline.
3. Notice two spawners spawning blood splats in separate directions.
4. Now run MAP01 on a server.
5. Join this server.
6. Notice only one spawner functioning properly.
Same thing happens in Zandronum 3.0.
No tags attached.
? argsnegative.wad (1,803) 2015-09-06 11:44
/tracker/file_download.php?file_id=1644&type=bug
Issue History
2015-09-06 11:44ZalewaNew Issue
2015-09-06 11:44ZalewaFile Added: argsnegative.wad
2015-09-06 12:29ZalewaAdditional Information Updatedbug_revision_view_page.php?rev_id=8040#r8040
2015-09-11 18:59DuskStatusnew => acknowledged
2015-09-11 18:59DuskStatusacknowledged => assigned
2015-09-11 18:59DuskAssigned To => Dusk
2015-09-11 18:59DuskStatusassigned => acknowledged
2015-09-17 15:23DuskStatusacknowledged => assigned
2015-09-17 16:09DuskNote Added: 0013526
2015-09-17 16:09DuskStatusassigned => needs review
2015-09-17 16:10DuskNote Edited: 0013526bug_revision_view_page.php?bugnote_id=13526#r8067
2015-10-04 14:13Torr SamahoNote Added: 0013591
2015-10-04 14:13Torr SamahoStatusneeds review => needs testing
2015-10-05 18:55ZalewaNote Added: 0013614
2015-10-06 16:22DuskStatusneeds testing => resolved
2015-10-06 16:22DuskFixed in Version => 3.0
2015-10-06 16:22DuskResolutionopen => fixed
2018-09-30 22:18Blzut3Statusresolved => closed

Notes
(0013526)
Dusk   
2015-09-17 16:09   
(edited on: 2015-09-17 16:10)
'https://bitbucket.org/Torr_Samaho/zandronum-stable/pull-requests/11 [^]'

The client was setting the thing arguments as 255 as instructed by the server. We need 4 byte length for thing arguments because UDMF and modern DECORATE are things now.

(0013591)
Torr Samaho   
2015-10-04 14:13   
I added your patch.
(0013614)
Zalewa   
2015-10-05 18:55   
Tested my example in 3.0-alpha-r151004-2012. Seems to be working fine.