[Freeciv-Dev] Re: (PR#2223) cma: server knowledge of island != client knowledge

["Raimar Falke via RT" <rt@xxxxxxxxxxxxxx>]

On Wed, Jan 08, 2003 at 09:47:15PM -0800, Anthony J. Stuckey via RT wrote:
> On Wed, Jan 08, 2003 at 10:44:47AM -0800, Raimar Falke via RT wrote:
> > 
> > [rfalke - Wed Oct 23 07:43:28 2002]:
> > 
> > The recently committed solution doesn't fix this issue. The reason
> > is that trade_between_cities uses the map knowledge of two players.
> > This is bad since the client only has this knowledge for itself.
> > The attached patch fixes this and also adds a comment.
> > With the patch this issue is resolved and you get an output of:
> >   2: Harstad (19) and Tromsø (9) -> 8
> >   2: Tromsø (9) and Harstad (19) -> 4
> > indicating that Harstad gets 8 trade from this route and Tromsø 
> > only 4.
> 
>       Yikes!
> 
>       How can this be a good thing?!?!?!?
>       Trade should be the same value at both ends.

Mike and the others decided over xmas that effects and traderoutes
should be based on player knowledge. While the committed code didn't
show the above different results for the two cities it also didn't fix
the issue which lead to this discussion: the above 2223. I fixed this
under the condition of player based knowledge. If you don't like this
you have to be on Davide side who preferred a solution where not the
player knowledge but the server knowledge (which is constant for all
players) decides.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
  Tank: So what do you need? Besides a miracle.
  Neo: Guns. Lots of guns.
    -- From The Matrix