Follow-up Comment #4, bug #16100 (project freeciv):
> Shouldn't unit_activity_assign_target() return boolean instead
> of void to tell if it was successful or not?
> unit_activity_handling_targeted() could use that information to
> decide if it should continue setting activity or not.
Also, I don't like how unit_activity_assign_target() in this case assigns
activity itself (name of the function no way implies that, on the contrary
name mentions that only certain "part" (target) of the activity information
is set). I recommend leaving that to caller, once it receives "failure"
return value from unit_activity_assign_target(). Another possibility is to
rename function (but I have no good suggestions what the name could be)
Reply to this item at:
Message sent via/by Gna!
Freeciv-dev mailing list