[Freeciv-Dev] unit_to_use_to_pillage

[David Pfitzner <dwp@xxxxxxxxxxxxxx>]

The recent patch to allow units to specify what to pillage
has a global var:

+struct unit *unit_to_use_to_pillage;

To be safe, shouldn't this be a unit id, in case the unit
gets killed between popping up the pillage dialog, and the 
user making a selection?  

(In which case the pointer would be wild, whereas looking 
up the unit id would tell you the unit doesn't exist any 
more.)

-- David