Bug 4999 - the marauder zap code is seriously bugged
Status: RESOLVED FIXED
Alias: None
Product: Tremulous
Classification: Unclassified
Component: Misc
Version: SVN HEAD
Hardware: All All
: P3 major
Assignee: Tim Angus
QA Contact: Tremulous Bugs
URL:
Depends on:
Blocks:
 
Reported: 2011-05-23 19:59 EDT by /dev/humancontroller
Modified: 2011-08-08 00:27:45 EDT
1 user (show)

See Also:


Attachments
uber advanced marauder zap fix (10.02 KB, patch)
2011-05-23 20:13 EDT, /dev/humancontroller

Description /dev/humancontroller 2011-05-23 19:59:11 EDT
- the zap chain range is box shaped, and the box width has an obscure size, ~232qunits
- the zap chains to secondary targets visible to the zapper, as opposed to targets visible from the primary target
- a buildable may only be zapped by 1 advanced marauder at a time
- the zap effect does not show up when the target dies due to a zap
- possibly other things i don't recall atm
Comment 1 /dev/humancontroller 2011-05-23 20:13:22 EDT
Created attachment 2736 [details]
uber advanced marauder zap fix

this patch addresses the mentioned issues.

as Norfenstein requested, all the zap ranges (first zap, zap chain) r now sphere shaped, having a radius of 200 qunits. 

furthermore, even though zaps do not chain through dead targets, a visual effect is shown when zapping dead buildables (doing 0 damage). the visual effect is removed instantly when the zapped target disappears, ie., a zapped player leaves the team, or a zapped buildable is deconstructed.
Comment 2 /dev/humancontroller 2011-05-23 20:22:14 EDT
even though it is now possible to chainzap a buildable with 2 advanced marauders at the same time (the appropriate damage is done), only 1 zap line shows up at a time. i believe is a rendering bug in the cgame module: i think the cgame assumes that a buildable is only chainzappable once at a time. this is a candidate for more fixing.
Comment 3 Chris "Lakitu7" Schwarz 2011-08-08 00:27:45 EDT
This looks like it was a lot of work, but it's done nicely. Thank you. Committed (with very few, trivial formatting modifications to fit <80 char width) at r2247.