[Freeciv-Dev] Re: DIRSTEP ?
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
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.
> 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 http://www.srcf.ucam.org/~gs234/
Is this "BOOZE"?