Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2004:
[Freeciv-Dev] (PR#10201) PATCH: generator characteristics evolution, bes
Home

[Freeciv-Dev] (PR#10201) PATCH: generator characteristics evolution, bes

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: mburda@xxxxxxxxx
Subject: [Freeciv-Dev] (PR#10201) PATCH: generator characteristics evolution, best handled temperature and relief.
From: "Mateusz Stefek" <mstefek@xxxxxxxxx>
Date: Wed, 22 Sep 2004 16:00:08 -0700
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=10201 >

> [mburda - Wed Sep 22 15:37:10 2004]:
This i getting really messy!
1) It's quite hard to remember all places where heightmap, tmap, placed
and characteristics should be created or freed.
Could you change them to local variables?
I imagine height map as a class with one public attribute:
  int* height_map
and with methods:
hmap_init
hmap_free_data
hmap_get_shore_level
hmap_get_mountains_level
hamp_normalize_poles
hmap_renormalize_poles
hmap_make_random_hmap
hmap_make_pseudofractal1_map

The same with other maps.
2) Move functions into proper files
  For example: terrain_is_too_high should be in height_map
probably renamed hmap_terrain_is_to_high
3) Why place terrain takes 3 characteristic_types? Can not it take their
sum?
--
mateusz



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