MantisBT - Zandronum
View Issue Details
0002247Zandronum[All Projects] Bugpublic2015-05-24 10:422018-09-30 22:19
Dark-Assassin 
Edward-san 
highmajorrandom
closedfixed 
3.0-beta 
3.03.0 
0002247: A_SetScale weirdness online
Online under certain conditions, A_SetScale will fail.

Tested with zandronum-3.0-r150523-1323-d946e45-windows
Load A_SetScaleTest.wad online
Summon actors ScaleTest ScaleTest2 ScaleTest3 ScaleTest4
Observe the differences
I only found it happens before or after A_Jump unless 2 separate frames both with >1 tics have passed. Length of those frames doesn't matter.

This breaks instant scale setting. Random or not.
No tags attached.
has duplicate 0002264closed Edward-san A_SetScale doesn't work online 
related to 0002265closed Torr Samaho A_SpawnItemEx flag SXF_TRANSFERSCALE doesn't work online 
? A_SetScaleTest.wad (917) 2015-05-24 10:42
/tracker/file_download.php?file_id=1498&type=bug
Issue History
2015-05-24 10:42Dark-AssassinNew Issue
2015-05-24 10:42Dark-AssassinFile Added: A_SetScaleTest.wad
2015-05-24 18:49DuskProduct Version => 3.0
2015-05-24 18:49DuskSummary3.0 Alpha - A_SetScale weirdness online => A_SetScale weirdness online
2015-05-24 19:37DuskProduct Version3.0 => 3.0-beta
2015-06-23 20:19Edward-sanNote Added: 0012777
2015-06-23 20:19Edward-sanAssigned To => Edward-san
2015-06-23 20:19Edward-sanStatusnew => needs review
2015-06-23 22:51Edward-sanNote Edited: 0012777bug_revision_view_page.php?bugnote_id=12777#r7525
2015-06-23 22:55DuskRelationship addedhas duplicate 0002264
2015-07-25 09:54Edward-sanNote Edited: 0012777bug_revision_view_page.php?bugnote_id=12777#r7774
2015-08-07 17:17Torr SamahoNote Added: 0013075
2015-08-07 17:17Torr SamahoStatusneeds review => feedback
2015-08-08 00:28Edward-sanNote Added: 0013078
2015-08-08 09:23Torr SamahoNote Added: 0013083
2015-08-08 10:14Edward-sanNote Added: 0013085
2015-08-08 10:20Edward-sanNote Edited: 0013085bug_revision_view_page.php?bugnote_id=13085#r7818
2015-08-08 15:26Torr SamahoRelationship addedrelated to 0002265
2015-08-08 17:11Torr SamahoNote Added: 0013106
2015-08-08 17:11Torr SamahoStatusfeedback => needs testing
2015-08-08 17:22cobaltTarget Version => 3.0
2015-08-08 17:22cobaltDescription Updatedbug_revision_view_page.php?rev_id=7826#r7826
2015-08-08 17:22cobaltSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=7828#r7828
2015-08-08 17:22cobaltAdditional Information Updatedbug_revision_view_page.php?rev_id=7830#r7830
2015-08-08 17:22cobaltNote Added: 0013107
2015-09-12 06:07Ru5tK1ngNote Added: 0013489
2015-09-12 11:39DuskStatusneeds testing => resolved
2015-09-12 11:39DuskFixed in Version => 3.0
2015-09-12 11:39DuskResolutionopen => fixed
2018-09-30 22:19Blzut3Statusresolved => closed

Notes
(0012777)
Edward-san   
2015-06-23 20:19   
(edited on: 2015-07-25 09:54)
Should be fixed with'https://bitbucket.org/crimsondusk/zandronum-sandbox-stable/commits/53216adaddc420c73a42aac197e51d82a49ce0e8 [^]' (patch made for stable, because I managed to port A_SetScale and A_SetMass to 2.x here:'https://bitbucket.org/crimsondusk/zandronum-sandbox-stable/commits/f5e0789eeba2d9a448e9d7e3b0900d107b1887c2 [^]').

Edit: fixed some code churn here:'https://bitbucket.org/crimsondusk/zandronum-sandbox-stable/commits/1d189a899246658e48884f30e2fce7b5655749de [^]' and'https://bitbucket.org/crimsondusk/zandronum-sandbox-stable/commits/4119b132329fc8d3433aaffc6882379864126d6d. [^]'

(0013075)
Torr Samaho   
2015-08-07 17:17   
Please recreate the patches in a single changeset and without backports based on 3.0's current head.
(0013078)
Edward-san   
2015-08-08 00:28   
Done, but please test if this works.
(0013083)
Torr Samaho   
2015-08-08 09:23   
Quote from Edward-san
Done

Thanks!

Quote from Edward-san
but please test if this works.

Wait, does this mean you didn't test this?
(0013085)
Edward-san   
2015-08-08 10:14   
(edited on: 2015-08-08 10:20)
I tested it before upstream git code upgrade, I didn't have the time to test it if this still works now, so this is why I didn't change from 'feedback' to 'needs review' after 0002247:0013078 .

(0013106)
Torr Samaho   
2015-08-08 17:11   
Thanks for your updated patch! I rebased, compressed and added it to the head.
(0013107)
cobalt   
2015-08-08 17:22   
Issue addressed by commit bfeb65caeb71: - Fixed: A_SetScale did not work online (addresses 2247 and 2264).
Committed by Edoardo Prezioso [edward-san] on Saturday 08 August 2015 17:32:40

Changes in files:

 src/cl_main.cpp | 43 +++++++++++++++++++++++++++++++++++++++++++
 src/network.h | 6 ++++++
 src/network_enums.h | 1 +
 src/sv_commands.cpp | 20 ++++++++++++++++++++
 src/sv_commands.h | 1 +
 src/sv_main.cpp | 12 ++++++++++++
 src/thingdef/thingdef_codeptr.cpp | 21 +++++++++++++++++++++
 7 files changed, 104 insertions(+), 0 deletions(-)

(0013489)
Ru5tK1ng   
2015-09-12 06:07   
I tried the test wad first with GZdoom 1.8.6 to verify zan 3.0 behavior was the same offline. Then I tested the wad online with 3.0 and the scale behavior between all 3 environments was the same.