Jump to content

  • Log In with Google      Sign In   
  • Create Account

We're offering banner ads on our site from just $5!

1. Details HERE. 2. GDNet+ Subscriptions HERE. 3. Ad upload HERE.


- - - - -

[AngelScript 2.26.3] Global property issues [Solved]


Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.

  • You cannot reply to this topic
5 replies to this topic

#1 SoulSharer   Members   -  Reputation: 270

Like
0Likes
Like

Posted 12 June 2013 - 03:33 AM

Hello friends,

I'm working on a game that requires intensive scripting (adventure game) and I experience some difficulties dealing with global properties.

It would be great if someone could help a fellow indie developer. Thanks.

 

Problem #1 (Solved - see below)

 

The problem is that in the example provided here: documentation link, we see that:

// Variables that should be accessible through the script.
CObject *g_object = 0;

// A function to register the global properties. 
void RegisterProperties(asIScriptEngine *engine)
{
 // Register variable where the script can store a handle to a CObject type. 
    // Assumes that the CObject type has been registered with the engine already as a reference type.
    r = engine->RegisterGlobalProperty("CObject @g_object", &g_object); assert( r >= 0 );
}

However it doesn't seem to work for me sad.png. I do it like this:

int r = 0;

GraphicsObject* go = ((GraphicsObject*)GetEntity("mooseCloth"));
r = engine->RegisterGlobalProperty("GraphicsObject @mooseCloth", &go); SE_ASSERT( r >= 0 );

go = ((GraphicsObject*)GetEntity("topCloth"));
r = engine->RegisterGlobalProperty("GraphicsObject @topCloth", &go); SE_ASSERT( r >= 0 );

go = ((GraphicsObject*)GetEntity("mooseHead"));
r = engine->RegisterGlobalProperty("GraphicsObject @mooseHead", &go); SE_ASSERT( r >= 0 );

Everything compiles without errors, but when I access this objects from script all of them point to the latest declaration "mooseHead". So for example if I go

topCloth.DoStuff();

instead of accessing topCloth it interferes with mooseHead. mellow.png

What am I doing wrong here?

 

One thing though, getting rid of this "@" for script variable declaration and "&" for passing the pointer to the object seems to solve that problem. But I'm not sure if it is a good way to go and if there any pitfall involved?

 

 

Problem #2 (Solved, see Andreas answer)

 

I want to declare a variable in a scope of module, so that when the game scene gets destroyed all the global variables are destroyed with it. However I didn't find a way to do that. There is a thing  CompileGlobalVar() but it doesn't allow me to pass the pointer to an object and there is no other function to do so (as far as I see), so I'm confused at that point. sad.png

There is no ability to destroy/unregister a global variable in the asIScriptEngine as well.

 

Thanks.

Yaroslav.


Edited by SoulSharer, 15 June 2013 - 03:03 AM.


Sponsor:

#2 SoulSharer   Members   -  Reputation: 270

Like
0Likes
Like

Posted 12 June 2013 - 02:12 PM

Oops, I'm sorry, it is in fact version 2.26.3 and I cant change the name of the topic.

Fixed, need to open full editor to change the name of the topic.


Edited by SoulSharer, 13 June 2013 - 02:42 PM.


#3 SoulSharer   Members   -  Reputation: 270

Like
0Likes
Like

Posted 13 June 2013 - 06:16 AM

Problem #2 - continuation

 

Object declarations are now added to the script source code before the compilation. And to initialize a variable I found out that I can get a pointer to a variable and it seems to work, but hell, it looks like a hack that asks for trouble, is it intended use?

int index = mod->GetGlobalVarIndexByName("topCloth");
GraphicsObject** adr = ((GraphicsObject**)mod->GetAddressOfGlobalVar(index));
*adr = (GraphicsObject*)(topClothObject);

Edited by SoulSharer, 13 June 2013 - 02:41 PM.


#4 SoulSharer   Members   -  Reputation: 270

Like
0Likes
Like

Posted 13 June 2013 - 12:13 PM

Problem #1 - Solution

 

I've got the answer on the first problem, thanks to Andreas, so I will post it here in case someone will face the same thing.

 

When you registered the properties with @ you were actually giving the address of the local pointer to the object. You then changed the object the pointer was referring to when registering the other properties, but you still registered the same address to the pointer with AngelScript. This caused all properties to refer to the same object.

Registering properties with @ should be used if the handle can be reassigned by the script to another object. Without the @ the property cannot be reassigned and will refer to the object you've registered throughout the execution.

 

Now it makes perfect sense.

Still not sure about problem #2 though. If I solved it right or is there another way which I didn't notice?


Edited by SoulSharer, 13 June 2013 - 12:16 PM.


#5 Andreas Jonsson   Moderators   -  Reputation: 3418

Like
1Likes
Like

Posted 13 June 2013 - 09:43 PM

You can use the GetAddressOfGlobalVar() to inspect and modify the global variables like you found out. Just make sure you update the reference counters of the objects correctly.

 

int index = mod->GetGlobalVarIndexByName("topCloth");
GraphicsObject** adr = ((GraphicsObject**)mod->GetAddressOfGlobalVar(index));
if( *adr )
 (*adr)->Release();
*adr = (GraphicsObject*)(topClothObject);
if( *adr )
 (*adr)->AddRef();

 

Regards,

Andreas


AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - Tower - free puzzle game

#6 SoulSharer   Members   -  Reputation: 270

Like
0Likes
Like

Posted 15 June 2013 - 03:01 AM

You can use the GetAddressOfGlobalVar() to inspect and modify the global variables like you found out. Just make sure you update the reference counters of the objects correctly.

int index = mod->GetGlobalVarIndexByName("topCloth");
GraphicsObject** adr = ((GraphicsObject**)mod->GetAddressOfGlobalVar(index));
if( *adr )
 (*adr)->Release();
*adr = (GraphicsObject*)(topClothObject);
if( *adr )
 (*adr)->AddRef();

Regards,

Andreas

 

Thanks again, as a suggestion I would add set property methods, so it will be more clear and maybe a little less error prone?

But anyway great stuff. :)






Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.



PARTNERS