30-03-2005 01:14									#1
	NoBarbarian Suffering from ads?

I'm currently adding a flag which means that that unit can't be picked as a barbarian to the code. Can anybody tell me if I'm duplicating something here and if not, whether adding it makes sense?

Can somebody revert the slic.y revision on SVN to r297? I made a mess there and I got lost trying to make it work again. Actually, only do this if the version there hasn't had ACTIVISION_ORIGINAL removed.

Last edited by tombom on 30-03-2005 at 02:02

	30-03-2005 01:48									#2
	Tired of ads?

nope, my cultureOnly only allowed for the possibility I think. But I recommend that you look into the BuildLists and instead look at the possibility of making a BarbarianBuildList and let the game pick offthere (instead off of the player build list)

	30-03-2005 12:02									#3
	Support Apolyton or Terrorists Win

I seem to have done something really odd to slic.y. Now reverting to the previous revision which I know worked doesn't work and I really have no idea how to make it work again. It only shows up if you use SlicDebug.

	30-03-2005 17:36									#4
	Support Apolyton, buy Civilization III: Complete

quote: Originally posted by tombom It only shows up if you use SlicDebug.

Where is it that it only shows up when you use SlicDebug?..

	30-03-2005 22:17									#5
	Full PM-box? Change here!

tut2_fun.slc, line 57 onwards has problems. The lines look perfectly normal, and they're all, as far as I can see, type declarations.

	31-03-2005 19:36									#6
	Increase Your PM Length

Looking at the diffs I'm somewhat confused as to what you have done. In you most recent change you deleted a line which you never added. I suspect that this may be the problem. Try:

svn diff -r 295:304 slic.y

to see what I mean. The typedef statement appears to have vanished - which would explain these errors.

Since I want to make some other alterations to slic.y, I'll put it back if you don't beat me to it.

Last edited by J Bytheway on 01-04-2005 at 02:14

	02-04-2005 01:03									#7
	Increase Your PM Length

tombom could you explain what you did to exclude barbarian units (its a nice add bytheway)

	02-04-2005 11:26									#8
	Inflate your Upload Space

quote: Originally posted by E tombom could you explain what you did to exclude barbarian units (its a nice add bytheway)

To start with I added Bit NoBarbarian to Unit.cdb. Then in barbarians.cpp there is a function ChooseUnitType which decides which unit type to make as the new barbarian. The code then goes through each unit and first decides whether that unit cannot be a barbarian. I just added my code which checked the unitdb for whether it had the flag NoBarbarian.

Here is the code, with the bit I added in bold.

code:


	for(i = 0; i < g_theUnitDB->NumRecords(); i++) {
		const UnitRecord *rec = g_theUnitDB->Get(i);
		if(rec->GetCantBuild())
			continue;

		if(!rec->GetMovementTypeLand())
			continue;
		
		if(rec->GetAttack() < 1)
			continue;

		if(g_exclusions && g_exclusions->IsUnitExcluded(i))
			continue;

		if(rec->GetNoBarbarian())
			continue;

I only added 3 lines

	03-04-2005 01:00									#9
	Support Apolyton, buy Civilization 2

nice tombom,
I suspected as much. Where did you put the bit NoBarbarian in the unit.cdb only because I've added quite a few things to the unit.cb and I'm not sure if the server would copy over yours.

	03-04-2005 01:14									#10
	Support Apolyton buy from Amazon

quote: Originally posted by E nice tombom, I suspected as much. Where did you put the bit NoBarbarian in the unit.cdb only because I've added quite a few things to the unit.cb and I'm not sure if the server would copy over yours.

In case of doubt the you are asked to resolve, conflicting changes by hand. However in the case of the *.cdb files there shouldn't be any conflicts.

-Martin