05-11-2003 01:55									#1
	Bugs in buildingutil.cpp ? Support Apolyton, buy Galactic Civilizations: Deluxe Edition

I was idly looking through the code and came upon the following lines in buildingutil.cpp. These three functions appear to be intended to return the maximum offense bonus for buildings present in a city. However, the body of the code seems to look for the wrong building characteristics for both water and ar improvements. AFAICS this implies that improvements such as costal forts or flak guns do not function as expected. The correction appears obvious. Thoughts?

code:
double buildingutil_GetOffenseBonusLand(const uint64 built_improvements)
{
	double best = 0;
	FOREACH_BUILT(GetOffenseBonusLand) {
		double cur;
		if(rec->GetOffenseBonusLand(cur) && cur > best)
			best = cur;
	}
	return best;
}

double buildingutil_GetOffenseBonusWater(const uint64 built_improvements)
{
	double best = 0;
	FOREACH_BUILT(GetOffenseBonusWater) {
		double cur;
		if(rec->GetOffenseBonusLand(cur) && cur > best)
			best = cur;
	}
	return best;
}

double buildingutil_GetOffenseBonusAir(const uint64 built_improvements)
{
	double best = 0;
	FOREACH_BUILT(GetOffenseBonusWater) {
		double cur;
		if(rec->GetOffenseBonusLand(cur) && cur > best)
			best = cur;
	}
	return best;
})

	05-11-2003 02:05									#2
	Support Apolyton

Yes it looks like some copy and paste work, so all you have to do is to find the right functions make shure all of them are implemented properly and plug them in.

-Martin

	05-11-2003 02:51									#3
	Support Apolyton, buy Civilization: The Boardgame

I replaced the code with the proper look up values. The code to actually return the values entered in buildings.txt looks correct. I havent tried to compile CTP2, yet. Here is the changed code:

code:
 

double buildingutil_GetOffenseBonusWater(const uint64 built_improvements)
{
	// Iterate through all the buildings in the city
	// and return the value of the MAXMIMUM bonus only,
	// This bonus is not additive
	double best = 0;
	FOREACH_BUILT(GetOffenseBonusWater) {
		double cur;
		if(rec->GetOffenseBonusWater(cur) && cur > best)
			best = cur;
	// original code below checked for the wrong bonus
	//	if(rec->GetOffenseBonusLand(cur) && cur > best)
	//		best = cur;
	}
	return best;
}

double buildingutil_GetOffenseBonusAir(const uint64 built_improvements)
{
	// Iterate through all the buildings in the city
	// and return the value of the MAXMIMUM bonus only,
	// This bonus is not additive
	double best = 0;
	FOREACH_BUILT(GetOffenseBonusAir) {
		double cur;
		if(rec->GetOffenseBonusAir(cur) && cur > best)
			best = cur;
	}
	return best;

	// original code below checked for the wrong bonus
	//	FOREACH_BUILT(GetOffenseBonusWater) {
	//		double cur;
	//		if(rec->GetOffenseBonusLand(cur) && cur > best)
	//			best = cur;
	//	}
	//	return best;
}
 

	05-11-2003 02:54									#4
	Support Apolyton, buy Galactic Civilizations: Deluxe Edition

nice one

	05-11-2003 03:11									#5
	Lose 30 kilos (of popups)

quote: Originally posted by The Big Mc nice one

Yup nice one I just would reduce the commented lines like this:

code:

double buildingutil_GetOffenseBonusAir(const uint64 built_improvements)
{
	// Iterate through all the buildings in the city
	// and return the value of the MAXMIMUM bonus only,
	// This bonus is not additive
	double best = 0;
	FOREACH_BUILT(GetOffenseBonusWater) {
		double cur;
		if(rec->GetOffenseBonusAir(cur) && cur > best)
	//	Original Code:
	//	if(rec->GetOffenseBonusLand(cur) && cur > best)
			best = cur;
	}
	return best;
}

And finally you should add your name to the fix so that we know in the case of doubt who to bla.. äh I mean who to give the credits for the fix.

-Martin

	05-11-2003 04:31									#6
	Inflate your Upload Space

Now youre finding AND fixing bugs i never knew existed, this is all going strangely quickly