View Issue Details

IDProjectCategoryView StatusLast Update
0000168Gameplay + OpenGL[All Projects] Bugpublic2017-02-07 08:38
ReporterMajor Cooke 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionunable to reproduce 
Summary0000168: D4D's Syphon Grenade Not Giving Armor
DescriptionI'm not sure what's happened but all I can say is, I didn't touch the syphon grenade anytime soon, nor did I touch the armor values. It would be appreciated if I could have some help investigating this since something caused the grenade to stop working correctly.

I don't know how long this has been that way but I just know I did nothing on my end to change that.
Steps To ReproduceLoad up D4D and start a game on Nightmare difficulty or greater. Throw a syphon grenade at some enemies and watch how your armor meter never rises.
TagsNo tags attached.

Relationships

Activities

Major Cooke

Major Cooke

2017-02-03 08:05

reporter   ~0000340

https://forum.zdoom.org/viewtopic.php?f=19&t=54843&p=967174#p967174

Download D4D here.
_mental_

_mental_

2017-02-04 02:41

developer   ~0000348

A little bit off-topic but still...
Bleeding edge version asserts at src/scripting/vm/vmbuilder.cpp:285:
Assertion failed: (locp->Tag == tag), function GetConstantAddress
It's somehow related to null values in the array declared in D4D/ZD4D/Pickups.txt:378
_mental_

_mental_

2017-02-04 02:49

developer   ~0000349

Forgot to mention the most important thing: it works for me with .pk3's from here using the latest commit d5962b2.
I tried on Doom II MAP01, Nightmare: two zombiemen gave me 40 armor points.
What's the commit you are using?
Major Cooke

Major Cooke

2017-02-04 07:31

reporter   ~0000357

The same one.

Are you using Bleeding Edge?
_mental_

_mental_

2017-02-04 07:46

developer   ~0000358

Yes, D4D.pk3 has been updated since I tried but the grenade still works with the new one.
I tried with own 64-bit macOS build and with 32-bit Windows devbuild from DRD, without .ini in both cases.
Major Cooke

Major Cooke

2017-02-04 08:36

reporter   ~0000360

Last edited: 2017-02-04 08:37

View 3 revisions

I'm using RelWithDebInfo currently. Hmm... I don't know what to say about the assertion thing.

Also Windows 10 Pro x64.

Graf Zahl

Graf Zahl

2017-02-04 09:58

administrator   ~0000368

The assert is a bug, thanks to some very shortsighted design in the VM with those stupid pointer tags. It's a perfectly harmless condition aside from that debug code that raises an alert. I have been thinking about removing that stuff anyway, as it inflates runtime data structures and other stuff for some very minor debugging aid during compiler development. Originally this had a lot more pointer tags but as it turned out the system was totally unusable because it couldn't always keep track of them.
Graf Zahl

Graf Zahl

2017-02-04 10:09

administrator   ~0000369

I'm sorry that I have to say it this bluntly, but you are essentially asking to have your code debugged. This is first and foremost your own responsibility to ensure that it all works as you wish.

This is simply not what the bug tracker is there for.

If you can pinpoint a specific cause that's engine related, please report back but I really need to work on other things than debugging code I do not even fully understand how it's supposed to work.

_mental_

_mental_

2017-02-04 10:47

developer   ~0000370

Just checked with 64-bit Windows RelWithDebInfo configuration and bleeding edge .pk3's downloaded a few minutes ago. It still works for me. No .ini, no other WADs, Doom II MAP01.
Major Cooke

Major Cooke

2017-02-04 10:56

reporter   ~0000371

It's strange because I never changed anything on my end, I was wondering if the armor scriptifying had something to do with it.

Also I forgot to mention it's sporadic when it decides to stop working. I'm trying to investigate it as thoroughly as possible.
Major Cooke

Major Cooke

2017-02-07 07:58

reporter   ~0000423

I cannot reproduce this anymore so go ahead and close it.

Issue History

Date Modified Username Field Change
2017-02-03 08:03 Major Cooke New Issue
2017-02-03 08:05 Major Cooke Note Added: 0000340
2017-02-04 02:41 _mental_ Note Added: 0000348
2017-02-04 02:49 _mental_ Note Added: 0000349
2017-02-04 07:31 Major Cooke Note Added: 0000357
2017-02-04 07:46 _mental_ Note Added: 0000358
2017-02-04 08:36 Major Cooke Note Added: 0000360
2017-02-04 08:37 Major Cooke Note Edited: 0000360 View Revisions
2017-02-04 08:37 Major Cooke Note Edited: 0000360 View Revisions
2017-02-04 09:58 Graf Zahl Note Added: 0000368
2017-02-04 10:09 Graf Zahl Status new => needs testing
2017-02-04 10:09 Graf Zahl Note Added: 0000369
2017-02-04 10:47 _mental_ Note Added: 0000370
2017-02-04 10:56 Major Cooke Note Added: 0000371
2017-02-07 07:58 Major Cooke Note Added: 0000423
2017-02-07 08:38 _mental_ Status needs testing => resolved
2017-02-07 08:38 _mental_ Resolution open => unable to reproduce