Threaded Mode | Linear Mode

***XuN*** · 01-25-2015 07:32 PM

The passable function is really INSANE, it have up to 8k lines depending on the arg given!! hence up to 4k checks.

As I can see you are checking the Terrains to see if you can walk upon them or not, right? what about selecting terrain types using serv.map(x,y,z).type instead of .terrain and discarding the ones you don't want? something like ...

Code:

local.terrain=<serv.map(<eval <local.x>>,<eval <local.y>>,<argv4>).type>

local.passable=<qval (<local.terrain>==t_water || <local.terrain>==t_rock || <local.terrain>==t_lava) ?0:1>

You can add more checks there, even create new terrain types just to create a 'direct' check instead of 4k of them.

I would even better suggest you to create one definition per valid terrain, something like:

Code:

[defname valid_Terrains]

terrain_valid_1=1

terrain_valid_2=1

...

terrain_valid_22=1

...

terrain_valid_16383=1

and checking it via

Code:

local.passable=<def0.terrain_valid_<serv.map(<eval <local.x>>,<eval <local.y>>,<argv4>).terrain>>

rather than using that big one function (but I prefer the first option).

Threaded Mode \| Linear Mode Sever goes unstable and crashes
Author	Message