search

Razish / japp

JA++ server and client modification for Jedi Academy
GNU General Public License v2.0
44 stars 30 forks source link

g_weaponDisable should remove ammo pickups #395

Closed Razish closed 8 years ago

Razish commented 8 years ago

issue seems related to trip mines etc being ammo_ entities

ensiform commented 8 years ago

It's actually related to checking for sabers only.

If you still allow melee and turrets then it's not technically saber only. It should be smart enough to disable ammo if all weapons that use an ammo type are disabled.

Razish commented 8 years ago

disable ammo if all weapons that use an ammo type are disabled

Agreed. It could be done on a per ammo type basis so it only keeps around the relevant ammo pickups.

ensiform commented 8 years ago

May want to still improve the saber only early out so it doesn't have to do even more checks for per ammo, though.

Razish commented 8 years ago

Not a performance issue, the check for spawning pickups only need to be done once.