empire/empserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

update budget produce: Count loaded civilians for TECH_POP

Browse source 
When option TECH_POP is enabled, the cost of tech increases when
civilian population exceeds 50000.  Only civilians in old-owned
sectors count.  This differs from education and happiness, where
civilians loaded on ships and land units count, too.

The update counts population for TECH_POP with count_pop().  This is
an extra pass over all sectors.

produce also uses count_pop(), once per tech center examined.
Wasteful.

budget avoids count_pop(): it uses tax()'s civilian count.  More
efficient, but the difference to the update is ugly.

Change TECH_POP to use the same civilian count as education and
happiness, i.e. count civilians on ships and land units, too.

This count is available in nat_budget[] in time for produce(): it's
computed by tax() and by ship and land unit maintenance.  So use it
there.  This takes care of the update and budget.  produce doesn't run
enough update code to do the same.  Keep calling count_pop() there.
Update it to match the update, and give it internal linkage.
Duplicating update's workings there is ugly, so mark it FIXME.

Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
This commit is contained in:
Markus Armbruster 2016-06-19 13:21:35 +02:00
parent 14f3bd9717
commit 4be4a2c540
6 changed files with 29 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

38
src/lib/commands/prod.c
View file

@ -37,27 +37,42 @@
#include <math.h>
#include "commands.h"
#include "item.h"
#include "land.h"
#include "optlist.h"
#include "product.h"
#include "ship.h"
#include "update.h"
static void prprod(struct sctstr *, double, double, char,
double, double, double, char[], int[], int[], int);
int
count_pop(int n)
static int
count_pop(void)
{
/* FIXME don't duplicate the update's workings, use its code */
int i;
int pop = 0;
struct sctstr *sp;
struct sctstr *sectp;
struct shpstr *sp;
struct lndstr *lp;
for (i = 0; NULL != (sp = getsectid(i)); i++) {
if (sp->sct_own != n)
continue;
if (sp->sct_oldown != n)
continue;
pop += sp->sct_item[I_CIVIL];
for (i = 0; (sectp = getsectid(i)); i++) {
if (sectp->sct_own == player->cnum) {
if (sectp->sct_own == sectp->sct_oldown)
pop += sectp->sct_item[I_CIVIL];
}
}
for (i = 0; (sp = getshipp(i)); i++) {
if (sp->shp_own == player->cnum)
pop += sp->shp_item[I_CIVIL];
}
for (i = 0; (lp = getlandp(i)); i++) {
if (lp->lnd_own == player->cnum)
pop += lp->lnd_item[I_CIVIL];
}
return pop;
}
@ -70,7 +85,7 @@ prod(void)
struct pchrstr *pp;
double p_e;
double prodeff;
int totpop;
int totpop = -1;
double cost;
int i;
int nsect;
@ -155,7 +170,8 @@ prod(void)
cost = take * pp->p_cost;
if (opt_TECH_POP) {
if (pp->p_level == NAT_TLEV) {
totpop = count_pop(sect.sct_own);
if (totpop < 0)
totpop = count_pop();
if (totpop > 50000)
cost *= totpop / 50000.0;
}