[Freeciv-Dev] Re: Changing interface for generate_warmap (PR#1108)

[Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>]

 --- Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> wrote: 
> On Thu, Dec 13, 2001 at 06:34:05PM +0000, Gregory Berkolaiko wrote:
> >  --- Jason Short <vze2zq63@xxxxxxxxxxx> wrote: 
> >> I don't understand what Raimar is saying.  Why will the current
> >> interface Gregory has not be extensible?  Just pass in a local
> >> warmap instead of NULL, and it'll use that variable instead.  Of
> >> course, you'll have to initialize it separately - but that's
> >> probably a good thing; 
> 
> >> if you have a warmap for each city you don't want to free and
> >> re-malloc each time you generate a new one.
> 
> Good point.
> 

So which interface should I be aiming for?

And do you think this patch is useful and will be accepted under certain
conditions (if so, specify the conditions)?

G.

__________________________________________________
Do You Yahoo!?
Everything you'll ever need on one web page
from News and Sport to Email and Music Charts
http://uk.my.yahoo.com