Bjond
Well-known member
Unsure what happens for an inventory update -- it could have been coded as a simple stack count decrement on up to a full DB push on each ranged shot. Consider that a ranged DPS character will typically fire 86/m at cap. That's a lot of essentially unnecessary inventory operations.
If inventory/ammo updates are more than a count decrement, perhaps consider making ALL ammo "returning". Then there's never any updating necessary for any ammunition. Could even rip out all ammo stack updates and assume it's returning (because it then would be). You'll likely want to do a little extra and make existing ammo non-stacking and update prices accordingly for vendor ammo, but that's about it -- a tiny change.
IMHO, this has the added benefit of also making quivers + RNG & CC ammo viable again. Swapping in ammo to break DR or apply the correct bane damage is rather pointless now due to stacking limitations and likely would be for most players even after the change. But, with all ammo "returning" at least the potential for using novel ammo exists.
If inventory/ammo updates are more than a count decrement, perhaps consider making ALL ammo "returning". Then there's never any updating necessary for any ammunition. Could even rip out all ammo stack updates and assume it's returning (because it then would be). You'll likely want to do a little extra and make existing ammo non-stacking and update prices accordingly for vendor ammo, but that's about it -- a tiny change.
IMHO, this has the added benefit of also making quivers + RNG & CC ammo viable again. Swapping in ammo to break DR or apply the correct bane damage is rather pointless now due to stacking limitations and likely would be for most players even after the change. But, with all ammo "returning" at least the potential for using novel ammo exists.