This is the mail archive of the
xconq7@sources.redhat.com
mailing list for the Xconq project.
Re: update material display
- From: Hans Ronne <hronne at comhem dot se>
- To: Peter Garrone <pgarrone at acay dot com dot au>
- Cc: xconq7 at sources dot redhat dot com
- Date: Mon, 6 Oct 2003 09:36:42 +0200
- Subject: Re: update material display
>Hi,
> I do have a few points with the recent materials update.
>
>1) The calculation is grossly inaccurate for bellum anyway, I cant
>figure out why advances seems correct, possibly because it has
>only advanced units for cities. The code from
>tkmain.c:update_material_display is as follows:
I'll check in the bug fix. Looking at the code again, all units should be
tested for is_active also, rahter than completed. Dead units don't produce
stuff.
>2) The capacity was removed. For Bellum, this is the most important
>parameter, so I have put it back. Here is my patch on the current code.
We would have to make it work correctly for all games, if we put it in. It
still does not handle treasuries.
Hans