Zandronum Chat @ irc.zandronum.com
#zandronum
Get the latest version: 3.0
Source Code

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002748Zandronum[All Projects] Bugpublic2016-06-05 05:472017-11-19 10:59
ReporterStrikerMan780 
Assigned ToEdward-san 
PrioritynormalSeveritymajorReproducibilityalways
Statusneeds testingResolutionopen 
PlatformMicrosoftOSWindowsOS VersionXP/Vista/7
Product Version3.0-beta 
Target Version3.1Fixed in Version 
Summary0002748: Changset 9556:54c5571372c0 Isn't supposed to affect A_GunFlash, but it does.
DescriptionZDoom introduced a change that makes the Muzzle Flashes for the stock weapons use 2 tics, to mimic vanilla Doom behavior, however, it wasn't supposed to affect A_GunFlash, which allows one to make 1-tic flashes.

When Zandronum introduced these changes, it got fucked up and affects A_GunFlash too, breaking some weapons in mods considerably. (No muzzle flash showing, or screwing up when codepointers fire.)

See this post and the reply:http://forum.zdoom.org/viewtopic.php?p=898285#p898285 [^]
Attached Files? file icon TestWad_BrokenFlash.wad [^] (191,789 bytes) 2017-11-13 17:06

- Relationships
child of 0002607needs testingEdward-san Weapon sync regression in 3.0 

-  Notes
User avatar (0015018)
WaTaKiD (updater)
2016-06-05 06:08
edited on: 2016-06-05 06:09

the 'repeated flash state' fix was disabled here:https://bitbucket.org/Torr_Samaho/zandronum/commits/2ad9d0af90838a40076af4e7367f9954ab303d9c [^]

due to causing a weapon sync regression, as seen here:http://zandronum.com/tracker/view.php?id=2607 [^]

User avatar (0015020)
StrikerMan780 (reporter)
2016-06-05 06:58

*sigh* Fuck me, always something.

Hopefully it can be fixed without disabling it soon. Quite a few recently created mods break because of this.
User avatar (0018873)
Edward-san (developer)
2017-11-13 16:43
edited on: 2017-11-13 16:43

Striker, could you upload an example with the A_GunFlash problem? I need to test the fix for ticket 0002607.

User avatar (0018874)
StrikerMan780 (reporter)
2017-11-13 16:50

Yeah, give me a little bit, and I'll put one together.
User avatar (0018875)
StrikerMan780 (reporter)
2017-11-13 17:07

Uploaded.
User avatar (0018876)
Edward-san (developer)
2017-11-13 17:55

Ok, the patch provided in 0002607:0018845 seems to work fine with the provided wad, both offline and online, but we need more tests to find out if anything else is broken.
User avatar (0018886)
Leonard (developer)
2017-11-16 16:46

I got to test this and even with the conditions mentioned in 0002607:0018833 the example wad clearly works with the fix (and doesn't without).
On that note, I don't think we should wait further before pulling these changes, the kind of testing Torr asks for here (i.e. real world/general testing) should ideally be provided by more frequent beta releases and not delay merges IMO.
User avatar (0018892)
Torr Samaho (administrator)
2017-11-19 10:59

I added the patch.
Quote from Leonard

On that note, I don't think we should wait further before pulling these changes, the kind of testing Torr asks for here (i.e. real world/general testing) should ideally be provided by more frequent beta releases and not delay merges IMO.

Just to clarify: I was not asking for real world testing to be done before the patch can be pulled. I was merely asking whether such testing already has been done to get an impression on which kind of testing baseline I can rely on to decide whether to pull.

Issue Community Support
Only registered users can voice their support. Click here to register, or here to log in.
Supporters: slowdensity
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2016-06-05 05:47 StrikerMan780 New Issue
2016-06-05 06:08 WaTaKiD Note Added: 0015018
2016-06-05 06:09 WaTaKiD Note Edited: 0015018 View Revisions
2016-06-05 06:58 StrikerMan780 Note Added: 0015020
2016-06-05 10:31 Dusk Relationship added child of 0002607
2016-12-24 22:43 Dusk Target Version => 3.0
2017-11-09 00:02 Leonard Status new => assigned
2017-11-09 00:02 Leonard Assigned To => Edward-san
2017-11-09 00:03 Leonard Target Version 3.0 => 3.1
2017-11-13 16:43 Edward-san Note Added: 0018873
2017-11-13 16:43 Edward-san Status assigned => feedback
2017-11-13 16:43 Edward-san Note Edited: 0018873 View Revisions
2017-11-13 16:50 StrikerMan780 Note Added: 0018874
2017-11-13 16:50 StrikerMan780 Status feedback => assigned
2017-11-13 17:06 StrikerMan780 File Added: TestWad_BrokenFlash.wad
2017-11-13 17:07 StrikerMan780 Note Added: 0018875
2017-11-13 17:55 Edward-san Note Added: 0018876
2017-11-13 17:55 Edward-san Status assigned => needs testing
2017-11-16 16:46 Leonard Note Added: 0018886
2017-11-19 10:59 Torr Samaho Note Added: 0018892






Questions or other issues? Contact Us.

Links


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker