Complete.Org: Mailing Lists: Archives: freeciv-dev: August 2001:
[Freeciv-Dev] Re: [PATCH] Corecleanup_07Part2 has been put in incoming

[Freeciv-Dev] Re: [PATCH] Corecleanup_07Part2 has been put in incoming

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [PATCH] Corecleanup_07Part2 has been put in incoming
From: Jason Dorje Short <jshort@xxxxxxxxxxxxx>
Date: Tue, 28 Aug 2001 16:53:46 -0400
Reply-to: jdorje@xxxxxxxxxxxxxxxxxxxxx

Raimar Falke wrote:
>On Tue, Aug 28, 2001 at 02:57:03PM -0400, Jason Dorje Short wrote:

> > Ross's patch has a greater scope than what you and I are considering;
>It may be possible that Ross has a greater scope but I consider the
>steps mentioned above the first ones.

I'd agree with that...but I don't think Ross will.

> > however it handles things differently than I would.
> >
> > As to the general issues:
> >
> > DIR_REVERSE has been replaced by 7-dir because Ross plans to change all
> > of the server code to use the rotational directional system, while
> > temporarily leaving the GUI using the vertical system (with magic
> > numbers).  I believe the better solution is to have all code continue to
> > use the vertical system (with no magic numbers) until all dependency
> > issues are resolved; however, I believe some of his applied patches have
> > already changed some DIR_REVERSE usages back to be 7-dir (for instance
> > in client/gui-*/mapview.c).  I feel strongly that this is not the
> > correct way to handle the problem; it makes fixing the GUI later even
> > harder.
>These changes weren't applied.

My mistake.  They were included in one of Ross's earlier patches, and a
search through the code did turn up 7-dir in those files.  I guess I had
patched my code at the time.

> > I will state again that non-real map positions should not be wrapped.
>I have nothing like this in mind.

No, but I believe others do.

> > Note that some of the patches you've applied have taken an opposite
> > approach to this plan.
>Please explain.

I'm specifically referring to the DIR_REVERSE(dir) => 7-dir changes
mentioned above, so never mind.


[Prev in Thread] Current Thread [Next in Thread]
  • [Freeciv-Dev] Re: [PATCH] Corecleanup_07Part2 has been put in incoming, Jason Dorje Short <=