[Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465)
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Den fre, 21 jul 2000 skrev jjm@xxxxxxxxxxxx:
> Full_Name: Jeff Mallatt
> Version: Latest CVS
> Distribution: Built from source
> Client: Both (or N/A)
> OS: Linux (Red Hat 5.0)
> Submission from: (NULL) (205.181.148.180)
> Submitted by: jjm
>
>
> I walked into an empty city, but it remained owned by the enemy!
>
> Problem is that move_unit() calls handle_move_consequences() [this is
> the only place that handle_move_consequences() is called] which calls
> handle_unit_enter_city() [this is the only place that
> handle_unit_enter_city() is called]. Okay, move_unit() is commented
> as performing "No checks whatsoever!". However, handle_unit_enter_city()
> performs a check! It checks to see if the attacker is at war with the
> target city. If not, it doesn't do the work to transfer the city. But,
> move_unit() continues to go ahead and move the attacking unit into the
> city.
Rather, you should not be able to take over the city of an enemy you are
not at war with. So the bug would be that the function that allowed you to
enter the city didn't check well enough.
But note that that check should not be removed! Allies can enter each
other's cities without taking them over, which is intentional and
documented. You are sure the players were not just allied?
-Thue
- [Freeciv-Dev] conquer city, owner doesn't change (PR#465), jjm, 2000/07/21
- [Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465),
Thue <=
- [Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465), Jeff Mallatt, 2000/07/23
- [Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465), Reinier Post, 2000/07/23
- [Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465), Thue, 2000/07/23
- [Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465), SamBC, 2000/07/23
- [Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465), Thue, 2000/07/23
- [Freeciv-Dev] Re: conquer city, owner doesn't change (PR#465), SamBC, 2000/07/23
|
|