clearemptyvars() crashes npc server.

[adam]

PHP Code:

function onCreated() {
  clearemptyvars(); // doesn't crash when used here
  
  this.test.var = "testing";
  echo( this.test.getDynamicVarNames() );
  
  this.test.var = "";
  echo( this.test.getDynamicVarNames() );
  
  //clearemptyvars();    // crash's when i swap it to here
  echo( this.test.getDynamicVarNames() );
  
} 


I'm not exactly sure what's going on here. It seems to crash when used alongside echo() Which is silly.

[cbk1994]

Just tested it and it crashed instantly when uncommenting the line you specified. Strange.

[xAndrewx]

Weird, version 5 right?

Shouldn't it be

obj.clearemptyvars();

either way, strange how it didn't crash on the first instance

[salesman]

Originally Posted by adam

PHP Code:

function onCreated() {
  clearemptyvars(); // doesn't crash when used here
  
  this.test.var = "testing";
  echo( this.test.getDynamicVarNames() );
  
  this.test.var = "";
  echo( this.test.getDynamicVarNames() );
  
  //clearemptyvars();    // crash's when i swap it to here
  echo( this.test.getDynamicVarNames() );
  
} 


I'm not exactly sure what's going on here. It seems to crash when used alongside echo() Which is silly.

I can confirm that this crashes the NPC Server, however, it has nothing to do with the echo(). Try setting temp.somevar = this.test.getDynamicVarNames() instead of echoing it, and the server will still crash.

[coreys]

Originally Posted by salesman

I can confirm that this crashes the NPC Server, however, it has nothing to do with the echo(). Try setting temp.somevar = this.test.getDynamicVarNames() instead of echoing it, and the server will still crash.

It might be some issue in the engine with threading? Race conditions can be nasty. Might be that you're trying to access this.test while it's locked. Though, why that would be done on a separate thread would be beyond me.

[adam]

PHP Code:

function onCreated() {
  temp.output = {};
  
  this.test.var = "testing";
  temp.output.add( this.test.getDynamicVarNames() );
  
  this.test.var = "";
  temp.output.add( this.test.getDynamicVarNames() );
  
  clearemptyvars(); 
  temp.output.add( this.test.getDynamicVarNames() );
  echo(temp.output);
} 


This example does not crash. And it also demonstrates the use of clearemptyvars() I was trying to demonstrate. Very useful.

[cbk1994]

Originally Posted by xAndrewx

Weird, version 5 right?

The NPC-server, not the client .

[adam]

I've actually never used this function clientside. So I dunno about that.

[xAndrewx]

Originally Posted by cbk1994

The NPC-server, not the client .

Oh duh, *peeks at thread*


My guess is that it's being used on a global level.

[WhiteDragon]

Well, my original speculation was that this.test was pointing to a memory address which didn't get cleaned up properly when you did clearemptyvars(), so accessing it would cause a seg fault.

But the fact that you can call getdynamicvars and add it to an array but not echo it seems a bit weird...


Maybe it has to do with casting the response of getdynamicvars to a string?

[Inverness]

Graal uses threading for scripts? News to me.