Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#546 closed defect (fixed)

improve/remove the check on unit height

Reported by: Bluestone Owned by: Bluestone
Priority: major Milestone:
Component: BA trunk Version:
Keywords: Cc:

Description

(maps can have arbitrarily high ground)

Change History (3)

comment:1 by beherith, 7 years ago

But in practice they max out at about 4000

comment:2 by Bluestone, 7 years ago

I think its better if I remove the check (it never existed before & were no problems) and then improve the gameend gadgetry to ignore crashing units?

comment:3 by Bluestone, 7 years ago

Resolution: fixed
Status: newclosed

Removed the check + prevented comblast from creating crashing aircraft - fixes #472 in a different way.

(An aircraft could still get blown high into the sky if it was crashing already and happened to be in just the spot when a com exploded beneath it; almost never happens)

Unfortunately its not possible to do a full fix for #472 because Spring.DestroyUnits doesn't affect crashing aircraft and Spring.GetUnitVelocity can't tell early enough if the unit is about to get blown high into the sky or not.

Last edited 7 years ago by Bluestone (previous) (diff)
Note: See TracTickets for help on using tickets.