search

Warzone2100 / old-trac-import

Archived Import of (old) Warzone 2100 Trac
0 stars 0 forks source link

bug or feature?: artillery stays assigned to sensor if nexus-turret "steals" it. assertion triggered. #2154

Closed wzdev-ci closed 12 years ago

wzdev-ci commented 14 years ago

keyword_assertion_nexus_turret_sensor resolution_fixed type_bug | by corvuscorax@...

I got a lot of those in the logs:

error   |12:12:18: [actionUpdateDroid] Assert in Warzone: action.c:2281 ((psDroid->psTarget->type ## OBJ_DROID || psDroid->psTarget->type OBJ_STRUCTURE) && (psDroid->psTarget->player == psDroid->player)), last script event: 'N/A'
error   |12:12:18: [actionUpdateDroid] DACTION_FIRESUPPORT: incorrect target type

What happened was, I had a lot of artillery units assigned to a sensor tank. But the sensor tank was "infected/stolen" by an enemy "nexus" tower, making it one of his.

Subsequently the - now enemy - sensor targeted one of my own nearby units.

And my artillery fired! Friendly fire!!! RUN!!!

Personally I don't think that behaviour really is a bug, but rather a really cool side effect of "taking over the right unit"

(though it would be cool if there'd be a quick way to re-assign a big amount of artillery units to a new sensor - currently one has to right-click them one-by one ;)

However the above assertion gets triggered, which indicates this "case" was not taken into account in action.c.

Somebody might want to go over it and fix at least the assertion to take this case into account.

Adding map+savegame

Issue migrated from trac:2154 at 2022-04-16 06:36:20 -0700

wzdev-ci commented 14 years ago

corvuscorax@... uploaded file nexus-arty.tgz (218.1 KiB)

map+savegame

wzdev-ci commented 14 years ago

KukY commented

Replying to [#2154 corvuscorax@…]:

(though it would be cool if there'd be a quick way to re-assign a big amount of artillery units to a new sensor - currently one has to right-click them one-by one ;) No need for one by one: you hold Ctrl and just select them all.

wzdev-ci commented 14 years ago

corvuscorax@... commented

Replying to Warzone2100/old-trac-import#2154 (comment:1):

Replying to [#2154 corvuscorax@…]:

(though it would be cool if there'd be a quick way to re-assign a big amount of artillery units to a new sensor - currently one has to right-click them one-by one ;) No need for one by one: you hold Ctrl and just select them all.

thanks for that one. and I thought I knew the interface >:)

wzdev-ci commented 14 years ago

anonymous commented

Replying to [#2154 corvuscorax@…]:

(though it would be cool if there'd be a quick way to re-assign a big amount of artillery units to a new sensor - currently one has to right-click them one-by one ;)

I do that by right click one and then use "CTRL+Z" (select all units of that type) then assign that selection to another unit/tower.

wzdev-ci commented 14 years ago

CorvusCorax commented

I wonder if that can also happen with mobile (or stationary) VTOL strike sensors.

probably yes ;)

Which bring us to the general dilemma of "close air support" >:>>>

wzdev-ci commented 12 years ago

Per Inge Mathisen changed status from new to closed

wzdev-ci commented 12 years ago

Per Inge Mathisen changed resolution from ` tofixed`

wzdev-ci commented 12 years ago

Per Inge Mathisen commented

Rewrite gift droid code. This should close a number of bugs in this code. Closes #3734. Closes #3733. Closes #3363. Closes #2154.