Random Chitchat 2012-2016

[xoft]

Slime loading, line 13: you're converting a pointer into a reference, I would expect that not to compile at all; then you're leaking the slime object if it doesn't load successfully. Use the same technique as in skeleton-loading - auto_ptr. I don't see why the skeleton loading should fail, no apparent problems there. Maybe push it to a branch so that I can give it a try with the debugger.

[tonibm19]

Mobs saviing!!

[tigerw]

Code #2:

It runs, I can sometimes even see the skeleton, but it immediately stops responding and after a while the deadlock detector asserts. I'll push to a branch soon, thanks!

[NiLSPACE]

I was wondering why the code didn't work until I discovered that GetMobType doesn't return the mob type of the mob. I found out that it returns a big number for example "24114938". http://puu.sh/5jZ2U.png

[tigerw]

Pushed to mobsaving branch, please examine

[xoft]

I was wondering why the code didn't work until I discovered that GetMobType doesn't return the mob type of the mob. I found out that it returns a big number for example "24114938". http://puu.sh/5jZ2U.png

Are you sure a_Entity is a cMonster descendant? Also, is that in debug or release? Release mode is sometimes wrong about variable values in the debugger, because of various optimizations.

Pushed to mobsaving branch, please examine

Later today, we have the in-laws coming for a visit so we're pretty busy right now

[NiLSPACE]

Are you sure a_Entity is a cMonster descendant? Also, is that in debug or release? Release mode is sometimes wrong about variable values in the debugger, because of various optimizations.

I use

if (!a_Entity->IsMob())
{
	return false;
}
if (((cMonster &)a_Entity).GetMobType() == mtWolf)
{

I tried it in debug and in release mode.

[xoft]

So is a_Entity a pointer or a reference? On line 1 you're using it as a pointer, on line 5 as a reference. If it's a pointer, cast it to cMonster * and it should work.

[xoft]

The mobsaving code was a mess, mixing references with pointers. I've fixed it and it seems to work; didn't test much though.

[tonibm19]

I tested it and it's working very well. Why don't merge it?