Problem with new python event

Hello,

First of all (as usual), I don't know if what I'm doing is correct, but I couldn't find another way to do it, I'm following what's in the game.

What I want is another event such as onUnitKilled, etc. I want onUnitUnload.
so I have copied CvEventManager.py to my mod's directory and added:


in the end of self.EventHandlerMap.

I also added the onUnitUnload function in this same file:


(it's commented because I thought this is what was wrong)

In the SDK, I have added:


in CvUnit::unload()

and in CvDLLEventReporterIFaceBase.h:


I compile it successfully, but the game crashes when unloading a unit. I don't know what I else I need to set but the logs in My Games don't show anything. I have init,mplog,pythonerr2,resmgr and xml logs but none of them have anything useful at all. in my mod's ini I have:



and chipotle.

that's one problem, the other is the crash. by using the debug DLL I have found out that it crashes right after the call to gDLL:

First-chance exception at 0x00b4a31d in Civ4BeyondSword.exe: 0xC0000005: Access violation reading location 0xd8ea86d8.
Unhandled exception at 0x00b4a31d in Civ4BeyondSword.exe: 0xC0000005: Access violation reading location 0xd8ea86d8.

Click to expand...

and that's all I know about the crash. solving both would be great

in the end what I want is doing something cool after an unit is unloaded, such checking for the plot and spawning a unit, stuff like that.

thanks.

salaminizer said:

In the SDK, I have added:

Code:

gDLL->getEventReporterIFace()->unitUnload(this);

Click to expand...

I believe that won't work because the event reporter has no inherent unitUnload() function and it's not part of the SDK (all we get is the interface description) so you can't simply add your own. What you want to do instead is use the genericEvent() function. Here's an example from CvGame::update():

In your case, you'd probably do something like this (untested):

Dresden said:

I believe that won't work because the event reporter has no inherent unitUnload() function and it's not part of the SDK (all we get is the interface description) so you can't simply add your own. What you want to do instead is use the genericEvent() function. Here's an example from CvGame::update():

Code:

		// sample generic event
		CyArgsList pyArgs;
		pyArgs.add(getTurnSlice());
		gDLL->getEventReporterIFace()->genericEvent("gameUpdate", pyArgs.makeFunctionArgs());

In your case, you'd probably do something like this (untested):

Code:

		CyArgsList pyArgs;
		pyArgs.add(this);
		gDLL->getEventReporterIFace()->genericEvent("unitUnload", pyArgs.makeFunctionArgs());

Click to expand...

it worked, thanks.
well, now that you say interface it makes sense I thought that just altering the .h didn't make much sense. next time I'll try READING the code instead of just "decoding" it...
thanks again

and about the logs, do you know what I'm doing wrong?

A direct SDK error often won't leave a trace in the logs. If the crash came as the result of a python function call, you'll get something in PythonErr.log, but in something like your case, debugging was the right call.