Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2003:
[Freeciv-Dev] Re: (PR#3450) square_iterate and unreal positions

[Freeciv-Dev] Re: (PR#3450) square_iterate and unreal positions

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: undisclosed-recipients:;
Subject: [Freeciv-Dev] Re: (PR#3450) square_iterate and unreal positions
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Mon, 17 Feb 2003 02:51:27 -0800
Reply-to: rt@xxxxxxxxxxxxxx

Raimar Falke wrote:
> On Mon, Feb 17, 2003 at 12:08:37AM -0800, Jason Short wrote:
>>For some reason square_iterate calls CHECK_MAP_POS on its center tile. 
>>This is a problem since it only shows up in debug mode and is not 
>>obvious to the programmer.  And I really don't know why it does that (I 
>>wrote the code for it, some time ago).
>>This CHECK_MAP_POS should probably be removed.
> Why? It is a check which tests if the args given to square_iterate are
> valid. We have the same checks in other functions/macros likesame_pos.

The definition of 'valid' may change.  In one situation, it is okay for 
the center of the square to be an unreal position.  In this case it is 
the (center_x+1,center_y+1) tile that determines the location of the square.

This is an untraditional usage of square_iterate; it is also reasonable 
to say square_iterate should not be used in this way.  But in that case 
we have the problem that developers are likely to misuse square_iterate.


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