Complete.Org: Mailing Lists: Archives: freeciv-dev: November 2001:
[Freeciv-Dev] Re: DIRSTEP ?

[Freeciv-Dev] Re: DIRSTEP ?

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxxxxxxx
Cc: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: DIRSTEP ?
From: Gaute B Strokkenes <gs234@xxxxxxxxx>
Date: Sun, 25 Nov 2001 02:46:42 +0000

On Mon, 12 Nov 2001, hawk@xxxxxxxxxxxxxxxxxxxxxxx wrote:
> On Mon, Nov 12, 2001 at 06:01:51PM +0000, Gaute B Strokkenes wrote:
>> On Mon, 12 Nov 2001, hawk@xxxxxxxxxxxxxxxxxxxxxxx wrote:
>> > On Sun, Nov 11, 2001 at 09:04:47PM +0000, Gaute B Strokkenes
>> > wrote:
>> >> 
>> >> Can someone remind me what this is supposed to do?  AFAICT it
>> >> only exists to obfuscate things.
>> > 
>> > We don't want direct access of the DIR_D[XY] arrays.
>> Why?
> Because we may change the implementation in the future. Mhh this
> isn't a good reason.

I do not see how DIRSTEP helps in that regard.  Perhaps you can

> In the current tree DIRSTEP is only used a few times. Which of these
> instance bother you?

All of them.  DIRSTEP() adds verbiage and complexity where none is
needed.  It frequently necessitates the use of temporary variables,
and the idiom of first loading a pair of offsets dx, dy into the
destination and then adding the original values x, y is
counterintuitive and confusing.

Big Gaute                     
Is this "BOOZE"?

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