(Possible) Breaking change: local variables references inside nested function definitions

d.healey

@danh said in (Possible) Breaking change: local variables references inside nested function definitions:

ok so just:

Yes, but it's no longer scoped now so you can change that comment ;)

Also hr does exist at init so you can't assign the value there, you have to do that within your function like you were before.

DanH

@d-healey said in (Possible) Breaking change: local variables references inside nested function definitions:

Also hr does exist at init so you can't assign the value there, you have to do that within your function like you were before.

with 'var'?

d.healey

@danh I'd say use a reg but if Christoph says a var is okay then var is ok. My point was you have reg origin = hr at the top of your script now, but hr doesn't exist until your FileSystem.browse call, so you can't assign it when you declare the origin variable at the top of your script you have to assign it after the call to browse just like you were with local origin = hr now you just put origin = hr. Or do what Christoph said.

DanH

This post is deleted!

DanH

@d-healey said in (Possible) Breaking change: local variables references inside nested function definitions:

Magic number alert!!! You should use hr.toString(hr.OnlyExtension)

Right, changing to the above makes the install process stop before the window appears allowing me to select the destination for the samples. So... select the .hr1 file and then nothing more...

d.healey

@danh Oo could be that the constant is bugged. I'm away from the computer for the next day so can't test, stick to your magic number for now :)

ustk

hr was already external, right? so why bother using another variable origin for the same thing in the first place?
It might be me, everyone around is telling me I need glasses... Arfff... 44... might be the time anyway...

d.healey

@ustk hr comes from the file browse call but the scope doesn't extend to the second browse call so it needs to be stored in a temporary variable

ustk

Just to say that I updated my custom branch, and all my var changed to local in the current project that is dependent on this very custom branch.
It's not a big deal to change back everything though as I remember this fix. But in the case it can be fixed it could help other people not to have this surprise, without knowing where it comes from :)

ustk

This post is deleted!

orange

@ustk Same here. I've used tons of local variables in all of my projects. Now this shit comes...

alt text

d.healey

@orange This problem was always there, it's just now you get a warning. It only applies to local variables in certain circumstances.

orange

@d-healey said in (Possible) Breaking change: local variables references inside nested function definitions:

@orange This problem was always there

My previous projects can't be opened with the latest develop branch.
But if I modify the local variables as var it works. So this issue is a new one.

d.healey

@orange

But if I modify the local variables as var it works. So this issue is a new one.

No it's an old issue, but now there is a new error so it might cause problems with opening old projects.

You don't need to change all locals to var - in fact that is a really bad idea. You only need to change the ones that are in nested functions. And I recommend not changing them to var because that pollutes the scriptwide namespace, I think it's better to create a reg object at the top of each namespace that can be used to hold the values - should I make a video about this?

orange

@d-healey Ok I will try with reg thanks.

So only modifying the places that the Hise gives the warnings is enough? Or even Hise doesn't give warnings, I should modify all nested functions?

d.healey

@orange

Ok I will try with reg thanks.

Just 1 reg at the top of the namespace.

I should modify all nested functions

Only nested functions (not inline functions) that are affected by this issue. The issue is that the variables go out of scope in certain circumstances. For example if you are calling something like Engine.showYesNoWindow() The callback function that you pass to that function will be triggered after any local variables have already been destroyed.

So it's basically anywhere you are using a callback function. A few examples would be:

Engine.showYesNoWindow()
Server.downloadFile()
File.extractZipFile()
FileSystem.browse()
FileSystem.browserForDirectory()

I am just trying to understand the case about what to do here. I have a script like below and I am getting Cannot define local variables outside of inline functions or callbacks. error in Hise - the current develop branch.

inline function onMYGENERICFUNCControl(component, value)
{
      local myLocalOne = Engine.getSystemTime();  
      local myLocalTwo = Engine.getVersion();
      local myLocalThree = Engine.showYesNoWindow(); 
        
    if(value)
    {
      // Execute a function

    }


    else
    {
      // Execute a function

    }
};

Content.getComponent("MYGENERICFUNC").setControlCallback(onMYGENERICFUNCControl);

So Should I transform the local variables with namespace & reg combination like this? Is this an effective usage?

namespace myNamespaceFunc
{	
	reg myLocalOne;
	reg myLocalTwo;
	reg myLocalThree;
			
	inline function onMYGENERICFUNCControl(component, value)
	{	
	    myLocalOne = Engine.getSystemTime();  
	    myLocalTwo = Engine.getVersion();
	    myLocalThree = Engine.showYesNoWindow(); 
	        	
	    if(value)
	    {
	      // Execute a function
	
	    }
	
		
	    else
	    {
	      // Execute a function
	
	    }
		
	};
		Content.getComponent("MYGENERICFUNC").setControlCallback(onMYGENERICFUNCControl);
	
};

d.healey

@Steve-Mohican You shouldn't be getting an error there, that is an inline function. Are you sure that is where the error is being generated?

@d-healey Yes I am getting that error exactly there.

I am also using a Server.callWithGET() inside the if statement and using these 3 variables in the if statement but I just trimmed the code to make it more clean

d.healey

@Steve-Mohican Aha now it's making more sense. The Server.callWithGET() function takes a callback function as its last parameter. Within this inner function you cannot declare local variables or use local variables declared outside of this function.

For this you should make a reg object at the top of your namespace called nest, and use this object in place of your local variables. For example:

Don't do this

namespace myNamespace
{
    inline function myFunc()
    {
        local myVar1 = 10;
        local myVar2 = 20;

        Server.callWithGET("google.com", {}, function()
        {
            Console.print(myVar1); // myVar1 doesn't exist here so you will get an error
        });
    }
}

Do this:

namespace myNamespace
{
    reg nest = {};

    inline function myFunc()
    {
        nest.myVar1 = 10;
        nest.myVar2 = 20;

        Server.callWithGET("google.com", {}, function()
        {
            Console.print(nest.myVar1);
        });
    }
}

@Christoph-Hart As a longer term solution is it possible to do something behind the scenes that allows local variables to exist within the inner functions? Maybe internally they are converted to reg variables...

(Possible) Breaking change: local variables references inside nested function definitions

22

1.8k

10.2k

89.1k