Complete.Org: Mailing Lists: Archives: freeciv-dev: August 2004:
[Freeciv-Dev] Re: (PR#9799) PATCH: rewrited adjust_map() 		Home

[Freeciv-Dev] Re: (PR#9799) PATCH: rewrited adjust_map()

[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: mburda@xxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#9799) PATCH: rewrited adjust_map()
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Tue, 31 Aug 2004 15:15:14 -0700
Reply-to: rt@xxxxxxxxxxx 
<URL: http://rt.freeciv.org/Ticket/Display.html?id=9799 >

Why is maxval a constant 1000 instead of just making it MAX_MAP_INDEX?

Why do you rename maxval and minval as max and min, when their use is 
unchanged?

These aren't major problems but I don't see why it can't be done better.

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