Complete.Org: Mailing Lists: Archives: freeciv-dev: March 2003:
[Freeciv-Dev] Re: (PR#3602) no_contact cleanup
Home

[Freeciv-Dev] Re: (PR#3602) no_contact cleanup

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: per@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#3602) no_contact cleanup
From: "Gregory Berkolaiko" <Gregory.Berkolaiko@xxxxxxxxxxxx>
Date: Tue, 4 Mar 2003 03:28:35 -0800
Reply-to: rt@xxxxxxxxxxxxxx

On Mon, 3 Mar 2003, Per I. Mathisen wrote:

> 
> This is a small cleanup of no_contact handling in the paradrop code. After
> the cleanup, the pplayer_no_contact function is no longer needed, and is
> removed.
> 
> I will commit later unless someone screams.

Scre-e-e-a-a-am!

Your patch is wrong: what if there are two units from different players on 
the tile?  What if there are other players units in the city?

You should use maybe_make_first_contact instead which handles all things 
properly.

G.




[Prev in Thread] Current Thread [Next in Thread]