Complete.Org: Mailing Lists: Archives: freeciv-dev: April 2000:
[Freeciv-Dev] Re: OO model for Freeciv?
Home

[Freeciv-Dev] Re: OO model for Freeciv?

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Dan Gohman <gohmandj@xxxxxxxxxxx>, freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: OO model for Freeciv?
From: Thue Janus Kristensen <thue@xxxxxxx>
Date: Sun, 30 Apr 2000 21:03:19 +0200

On Sun, 30 Apr 2000, Dan Gohman wrote:
> On Sun, Apr 30, 2000 at 07:35:12PM +0100, Sam BC wrote:
> > > -----Original Message-----
> > > From: Lalo [mailto:lalo@xxxxxxxxxx]On Behalf Of Lalo Martins
> > >
> > > If there was, even remotely, the chance that Freeciv could be
> > > upgraded to C++, it could be done by a separate group of madmen
> > > in a branch, and when ready, tested for performance. I for one
> > > wouldn't mind working on such a project and then throwing all
> > > my work in the trashcan if it doesn't turn out better than the
> > > C version.
> > 
> > I'm no genius, but I'd work on such a project as well. Translating from C to
> > C++ rewriting code to be essentially the same algorithmically but with a C++
> > implementation should not be too hard work for a small group of people
> > familiar with both languages...
> 
> Indeed it is not difficult. The hard part I'm finding is keeping the two
> versions in sync. I first started playing with this a while ago, and it was
> going well, but then the fog-of-war patch got integrated to cvs around
> the same time as I was unable to work on freeciv for about two weeks.
> I had made enough changes such that I had to merge everything into my C++
> version by hand (looking at the diff and then editing files manually). A
> very long task this is.

There are two more really big patches coming up, rulese and pacts. Have
fun... - Maybe you should time your activity to around a feature freeze?

Is there any reason to change the old code? It compiles as C++ I would
think (though I am not an expert at all). Switching to C++ would just make
the C++ features available for new patches. If it works don't fix it :)

-Thue



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