Anonymous | Login | Signup for a new account | 2024-04-23 06:16 UTC |
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0002978 | Zandronum | [All Projects] Bug | public | 2017-01-09 18:14 | 2018-09-30 21:44 | ||||
Reporter | Borg | ||||||||
Assigned To | Edward-san | ||||||||
Priority | normal | Severity | minor | Reproducibility | N/A | ||||
Status | closed | Resolution | fixed | ||||||
Platform | Source Code | OS | OS Version | ||||||
Product Version | 2.1 | ||||||||
Target Version | 3.0 | Fixed in Version | 3.0 | ||||||
Summary | 0002978: DamageFactor calculation executed twice for Armors | ||||||||
Description | After doing some modding with multiple Protection artifacts, I found a problem with stacking. It seems when 2 or more Protection are active, they start to stack badly and become much more strong that any single of them. After brief peek to source code, I found this strange thing: src/g_shared/a_armor.cpp,178: // Once the armor has absorbed its part of the damage, then apply its damage factor, if any, to the player Read up the whole block, with is executed twice (copy&paste). Looks like a bug to me, is it intended? | ||||||||
Additional Information | I would inspect other code around DamageFactor as well. As said, stacking Protection Inventory items greeat odd efect of very strong Protection. | ||||||||
Attached Files | |||||||||
Notes | |
(0016629) Dusk (developer) 2017-01-09 22:01 |
Looks like a merge error. A very old one too, as 2.1 also exhibits it. |
(0016631) Edward-san (developer) 2017-01-10 10:19 |
'https://bitbucket.org/zandronum/zandronum-sandbox/commits/187adb8ccddd380a6d122b5aecad622ff0ccac25 [^]' In 2.1, the two copies are identical, while in 3.0 they diverged because of the code upgrade (zdoom svn r3469, added in zandronum with changeset 6bbe67e64b39a951 ), so removing the second copy is the right way. |
(0016636) Torr Samaho (administrator) 2017-01-15 08:55 |
I added your patch. |
(0016707) Ru5tK1ng (updater) 2017-01-28 17:48 |
Can we get an example wad to correctly test the fix? |
(0017164) Combinebobnt (reporter) 2017-04-16 22:59 |
yea def are gonna need an example wad i tested stacking armor + 1 or 2 powerprotection items and nothing seemed out of the ordinary in both 2.1.2 and r170416-0710. Each protection was 0.5 damage and it worked the same in both ports. |
(0017193) Borg (reporter) 2017-04-18 07:39 |
Testing this is not obvious. I had 2 powerprotection items on my mod and it was all okey on single player test map. Things start to work strange on multiplayer... For example, suprised by HexaZombie.. I wasnt even damaged from his full blast hexa... After some more online testing then, I noticed that sometimes damage is very very low or nullified... After switching to ACS and APROP_DamageFactor things work good now. |
(0017356) Ru5tK1ng (updater) 2017-04-27 01:42 |
So is your issue now solved? It's hard to understand without an example wad. |
(0017740) Ru5tK1ng (updater) 2017-05-23 01:20 |
Marking as resolved since 'things work good now' according to the author. |
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: | Combinebobnt |
Issue History | |||
Date Modified | Username | Field | Change |
2017-01-09 18:14 | Borg | New Issue | |
2017-01-09 22:01 | Dusk | Note Added: 0016629 | |
2017-01-10 10:19 | Edward-san | Note Added: 0016631 | |
2017-01-10 10:19 | Edward-san | Assigned To | => Edward-san |
2017-01-10 10:19 | Edward-san | Status | new => needs review |
2017-01-15 08:55 | Torr Samaho | Note Added: 0016636 | |
2017-01-15 08:56 | Torr Samaho | Product Version | 4.0-beta => 2.1 |
2017-01-15 08:56 | Torr Samaho | Target Version | => 3.0 |
2017-01-15 08:56 | Torr Samaho | Status | needs review => needs testing |
2017-01-28 17:48 | Ru5tK1ng | Note Added: 0016707 | |
2017-04-16 22:59 | Combinebobnt | Note Added: 0017164 | |
2017-04-18 07:39 | Borg | Note Added: 0017193 | |
2017-04-27 01:42 | Ru5tK1ng | Note Added: 0017356 | |
2017-05-23 01:20 | Ru5tK1ng | Note Added: 0017740 | |
2017-05-23 01:20 | Ru5tK1ng | Status | needs testing => resolved |
2017-05-23 01:20 | Ru5tK1ng | Resolution | open => fixed |
2017-05-23 01:20 | Ru5tK1ng | Fixed in Version | => 3.0 |
2018-09-30 21:44 | Blzut3 | Status | resolved => closed |
Copyright © 2000 - 2024 MantisBT Team |