Zandronum Chat on our Discord Server Get the latest version: 3.2.1
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000734Zandronum[All Projects] Bugpublic2012-03-28 11:042018-09-30 21:24
Reporterunknownna 
Assigned ToDusk 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version98d 
Target VersionFixed in Version1.1 
Summary0000734: Incorrect ammo values in demos.
DescriptionI pick up a RL in the demo. The ammo counter displays "2/ 50", but when watching the demo it displays "4/ 50".
Additional InformationDemo recorded with 3417.
Attached Files? file icon 2012.03.28_12.56.55_doom2.cld [^] (28,165 bytes) 2012-03-28 11:04

- Relationships

-  Notes
User avatar (0004556)
Dusk (developer)
2012-09-01 17:24

 Confirmed in 1.0
User avatar (0004557)
Dusk (developer)
2012-09-01 17:39

 Heh, too easy P:
The a_weapons.cpp code lacked demo checks almost altogether. I guess this might fix some other weapon/demo bugs as well.
Fixed: Clients appeared to get double ammo from weapons while playing demos.
.. unless I'm missing something here and the demo checks were left out for a reason?
User avatar (0004572)
Torr Samaho (administrator)
2012-09-02 15:15

 Many of the demo checks were left out for a reason: All the blocks containing CLIENTCOMMANDS_WeaponSelect should never be called when playing back a demo.

I think your change in AWeapon::PickupForAmmo and the first change in AWeapon::AttachToOwner are correct though.
User avatar (0004683)
Torr Samaho (administrator)
2012-09-16 07:43

 Since it's been a while and the necessary adjustments are straight forward, I went ahead and revised the patch myself.
User avatar (0004757)
Watermelon (developer)
2012-09-23 17:19

 Should I get the latest build from bitbucket to test this?
User avatar (0004764)
Torr Samaho (administrator)
2012-09-23 18:30

 You can use this binary to test this.
User avatar (0004783)
unknownna (updater)
2012-09-23 20:46

 This works properly now.

Issue Community Support
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: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2012-03-28 11:04 unknownna New Issue
2012-03-28 11:04 unknownna File Added: 2012.03.28_12.56.55_doom2.cld
2012-03-28 11:04 unknownna Status new => confirmed
2012-06-09 13:22 Torr Samaho Category General => Bug
2012-09-01 17:24 Dusk Assigned To => Dusk
2012-09-01 17:24 Dusk Status confirmed => assigned
2012-09-01 17:24 Dusk Note Added: 0004556
2012-09-01 17:39 Dusk Note Added: 0004557
2012-09-01 17:39 Dusk Status assigned => needs review
2012-09-02 15:15 Torr Samaho Note Added: 0004572
2012-09-02 15:16 Torr Samaho Status needs review => feedback
2012-09-02 15:18 Dusk Status feedback => assigned
2012-09-16 07:43 Torr Samaho Note Added: 0004683
2012-09-16 07:44 Torr Samaho Status assigned => needs testing
2012-09-23 17:19 Watermelon Note Added: 0004757
2012-09-23 18:30 Torr Samaho Note Added: 0004764
2012-09-23 20:46 unknownna Note Added: 0004783
2012-09-23 21:54 Dusk Status needs testing => resolved
2012-09-23 21:54 Dusk Fixed in Version => 1.1
2012-09-23 21:54 Dusk Resolution open => fixed
2018-09-30 21:24 Blzut3 Status resolved => closed




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2025 MantisBT Team
Powered by Mantis Bugtracker