Can't reference inline function parameters in nested function body

[HISEnberg]

@d-healey Thanks for clarifying that terminology was a bit of a mystery to me, I wasn't aware of the factory method I have some homework to do!

[d.healey]

@HISEnberg

These are good references

Scripting Best Practices

This is a collection of useful tricks which will be extended over time. I divided them into two categories: Interface Design and MIDI Processing. Interface d...

Forum (forum.hise.audio)

HISE | Scripting | HISE Additions

A summary of all additions made to standard Javascript

(docs.hise.dev)

[JulesV]

@Chazrox @HISEnberg @d-healey
Thanks for the tips, guys. Very useful.

My aim was not to print the word "label". I asked how I could integrate the label variable into the ComboBox LAF.

inline function cbFctry(name, label, x, y, w, h)

So the label variable is "Types" here.

As previously mentioned, when I use this: g.drawAlignedText(label, [a[0], 28, a[2], 20], "centred");

I see: Can't reference inline function parameters in nested function body error.

[Chazrox]

@JulesV That was the second option I gave you. Create an object outside of your function and reference it within your function script. I also suggested using 'local' within your function to create the object.

[d.healey]

@JulesV The error there means that label is a parameter of your inline function but it's not in the scope of your laf function which is a different function.

What you can do instead is declare a label variable in the namespace, set this from within the inline function and reference it in the laf function.

[Chazrox]

This post is deleted!

[JulesV]

@d-healey said in Can't reference inline function parameters in nested function body:

What you can do instead is declare a label variable in the namespace, set this from within the inline function and reference it in the laf function.

Is this what you mean?

HiseSnippet 1297.3ocsV09aSbCF2WaOzxsWDHsuNIq7oKnakjzj9BcHnIsg0Qaoh.EjpPHGeN47hicjOGZi.9592a+gflz9Of83y2kjBgwT0vsIw9wOu7yOuYepVQYooJMxK3oSGyPdeme2oRSR6DBWhNbej2O3eLI0vzXGoVSGSRSYwHOuUenkfWo0PYi+99sHBhjxlSBgNSwori3i3l4TO8AOhKDcHwrmxGs.2MdvgTkrsRnl.3YU+pnwD5Px.1IDKaq3i9URZBx619MquAsQ7Vw0q2bmMojMY6zmzuesMhazXys2o1F6PZr8lrpMQd23fXtQo6ZHFVJxasVp3ocSTWHcF3LdJumfYWTC0ErribGkH1dDsTQsS3h3SKbToHPKmN2sspys8i9Gyi4ynO28cyrMvykXQGn2JWEdqdE3UaQ3UcA3sDH4s.jVyAoa42kp4iMy2whmu0+PIDM6Sf3zhPwwKZk+bM+1JfCoY8QjgrNZXwLIB2nZ0Hb8pUqrafc.gqTCt8nV36goZFbHZ2ZcZuNTidZXYfd4HbYKZRgIa2LBuE7o1FMhvalqh6bGba0ndpVpKwARHNmBwb1LkE7lBi7ZhFK5I.CUt7tkBJYEsz8Vx.2NSV7STSjPPbt5u2RGAbofKY39SjTCWIwEv2BlHrfziIhvWFgmFguHBmTI3Mf0EJJQfGBnovYQhiKrTtnVYfCYoBNRY1PwXl1vYofeczu08wm33MnzaBJUp7E7XSR46B1wtJgwGjXfkIQAAkdmUU1+bNgLbYoDT.Fs87F2t0Q60YAX4bjGY43HkZ3dx3NLlHrh0EtfDqqYC31p7N4tgvxwZxEEmHH5U3eBGDgU898JNHaiJDvb.k0Ifo1Mml4xbhF1kFKQ.lv.9BB3sdH76.q+vUqG59Ic8K3lj8DiSHgVQ4F1H2N0hvUWuYEqG.jqOz7HO39DF0PjCDrvyg7R3ex40eIjft0KsRTqRgkAidFQLgkiNZM.cY1HQ8Zny1aeaFVoBNcHKtB99yfty936lQn2.2xc+H3SyLDFFYT6.t9vx6o4DA32BqUu.3VO5dB9.IK9oftCMPFx4jyq5.aS7sAfO6HXoVlBgPMKtbkcsZOHyA5NNGYi+3eFuTECoHyzb8s+bprjy4jmXMzB8OIOYgTDGaZlYhVhGZW7trOtwrxGk7Dkg8XYHTnjsO9i2pe+ktmMiUqDBldoaaupP+uIXnbxndLcDzo.hzyXDZGd0dr9e9drKdE.0U.s.iJ4gRt4wiYxO2ECn7pNa+3bTArZx5Fey7twEkTHNz102G5Qhx.77qAu4CPO6v8IFRgZ.MNuwAPae1qgKUcM4K4uOKcnQMNi2QiURq88B9hF8RXh2y.yM0N4QvjQjLZ9vzrFQfxuwejeMtqWjkxujSwVcBf4m7eN3V0AGPzljfmykwAc3ZVvyUp3.TQEiUvO.iBAmS88u+u9f6QAex8WvMmp3IBh4pWmZezQ9FP7+J2gYumRlxMSW7QI+ucG6+UHdK+S4FZxxw3JKAiPb9qAFyeYx26eP+9Pex4.bM+Nu3qyyPPPeYCWN3XhQysYSmLYTWHRSYf0kRlvV+4shsRwstpcs0CzkIiyVXSSx2rlcsW9l0J1DxSoZ0qnt5K6ae9lLJ.ljYuSrD7fUXMt1GWXAo3vSwdEkdUU8IBV+5J3FWWAabcEr40UvMutBt00Uvs+xBZeo7dSLpQtxFD53SOHqQnm2ARBjAlksh9GQ5QfY.

[d.healey]

@JulesV Yes that's it, it can't be a const though because you can't change the value of a const. You can use a reg instead.

[JulesV]

@d-healey

reg variant can only be used inside the inline function. But this time there is another problem, when we want to clone the ComboBox, the last variable used is applied to all ComboBoxes.

In the below image, one CB should be "Types", the other one should be "Styles".

HiseSnippet 1325.3ocyWk0aabCDlqs2fpsGHAnu1BB8jTfpizZIeDWiDKYqT2Z6XDklT.if.pcozxJJRAtT1VHwu1+rAEnO2WZmYOjjikaRMZ.J00xgywGmY3LTmXzA73Xsg3387Ii3DmuvsyDkMpUDSnHGrGw4qbOhEa4FZJolSFwhi4gDGmkeBRvovJjjwe7nlLISEvmQhPdgVDvOTLTXmQ8jG+SBorMKj+bwv43t9iOHPqZok5w.dV1sJYDKX.qO+XFx1RtjefEGQbtuaC+0BpGtQnueisVOfsNeqdrd8psVX85qu4V0VaKV8MWmWsAw4N6GJrZSGKyxiINqzTGNoSj9bUpAdgHVzUxwI0Hc.KmRtsVFhaQjJoUjPFdRtiJl.Z4jYtskScaes6QhPwT5ybe2MYA5LIl2A5rzUg2xWAd0lGdUmCdK.RNyAoURgz8b6DXDiryVAwym6dfBhl8XPbZdnjxKYIGW2VZfCkc0grA71FXxTIJUuZ0JT+pUKusGNfvUrk1ZXyZzcnAFNrKZ0b0ftsCrlIkJhKTrBsHhmX3g0.g2.9Tas5UnqiJYpB7uIE3iJnichLQC9MtlJ77dvCnszC6papuf5ofbkXHugOUcduwq.xTgcVvf1JgK5yziUPHelh1YgCOgRJTbZuwp.qPqn4PEMaEpj0kKqPunBcRE54UnQk8dSAuBRc.SRG.awbWKKLL2RYhhx.6lB4bDyw.2HtwJ3wPTX3O14oGmxqWAXGUn34hPaTwGB1AmEwE8irvznJddEtDUE9xv6SkckfoSvFR0KGPFbOG1p4g61dNnk51ND43Psdvtpv1btrT4sQsMShUAMKv5BsybEkJFZXmmuqfXUtOpT+JTc2esbJrOiYnLvb.kUYfo1Nil8hLhV9EVjH.SX.eAg2lOA9sO5SRqNTJ8m3UOWXi1UNJhUBEUX4CSWoVEZ0UaTF8Bfb8fxMYA3mwCrLUeIuzoPVD7lcp+qf7pMdEJQsx4VFL5KXxw7LzEfI3I1HReFTK7suMAqARQv.dXY5ilB8T6SeXBgt8Smt86A+fDCQgQB01fquTwcMBlD7akp4mCbzitqTzWwCeNn6RVHK4T1oUSAaC58AfOcKfTKF.gPCOrX4sQs6k3.S2NGhwe52QWnhgTjoZ1eyaRkERcNYIWCPnes7j4RQRYyvsiMJ5.bxkIeRGSOBoUGqs7mpJUN4j5kdz2eod8V3ZXFqQKkbyBWFatX9mDrjZ7vtbSE5YXjdJiPAzqVU18lqJOeSifzCPywnVcfRXe5Ht5lZkPxN0AUvWJCU.q1j522Mq9c9QJh.JTeGWrnJIAwy25k7wKs+Bj9mOXOlkg8Qx.zrZO.s83mAcwS6pTvcOd7.qdTBhGNRqP3638gw7Efmz4aA6MAenM7vPVBMW3wjhYf1uyukcwgz5YHkuOiBd5FPy239RHrX71mYrQduTnB8ZKLbuWp0gdj7Sbnf+ELxEbF028teOgJ4eC58SQ+e9+Gze898vMMzgikL6Uu9AdIsrEfr+qzyG6qqhE1IymK7e1cR9Xg38bOQXChVLFWZAXDOr7I.iY2j6Kc2uWOnKwL.tha6e4Sy01HPWIqP0+Hl0HvroiGOrCDoC3f0UJtDq93rDlrlNuJNG8.c3pvjIXZR1h0v4NYKVKeQHOMvnecPZ8A7theVBE.Spj6UW.tfOLmdspJPJNb00WGDbUUcMA8usBt1sUv52VAabaEb8aqfabaEbyOrf3+rX2wV8vziMDxQmreRcbGm8ULHCLIak72X3cOCD

const CmB1 = createCB.cbFctry("CmB1", "Types", 30, 70, 134, 60);
const CmB2 = createCB.cbFctry("CmB2", "Styles", 250, 70, 134, 60);


// ComboBox 
namespace createCB
{
	

//	==================== Create Rounded ComboBox ======================
inline function cbFctry(name, label, x, y, w, h)
{	
	local k = Content.addComboBox(name, x, y);
	Content.setPropertiesFromJSON(name,
	{
		"width": w,
		"height": h,

	});
	
	
	reg lbl = label;
	


	local rounddCBLAF = Content.createLocalLookAndFeel();	
	rounddCBLAF.registerFunction("drawComboBox", function(g, obj)
	{
		var a = obj.area;
		var tx = obj.text;
		

				
		// BG
		g.setColour(Colours.withAlpha(obj.itemColour1, 0.5));
		g.fillRoundedRectangle([0, 0, a[2], 27], 0.1);
		

		// Value
		var c1 = (obj.hover || obj.clicked) ? obj.textColour : obj.bgColour;
		g.setColour(c1);
	    g.setFont("Arial", (12));
		g.drawAlignedText(tx, [a[0], 0.15 * 27, a[2], 20], "centred");    
		
		
		// Label - 
		g.drawAlignedText(lbl, [a[0], 28, a[2], 20], "centred");	
		

	});
	
	k.setLocalLookAndFeel(rounddCBLAF);
	
	return k;
	
}
	
}

[d.healey]

@JulesV said in Can't reference inline function parameters in nested function body:

reg variant can only be used inside the inline function.

Use local within inline functions. reg is for on init and namespaces.

However looking at your snippet it seems that the variable isn't visible within the laf function either.

So here's a better way to do it. Use the ID of the combo box itself, instead of calling them CmB1 and CmB2 we call it cmb followed by the label we want, e.g. cmbStyles. Then in the laf function we can remove the cmb part to get the label.

HiseSnippet 1327.3ocyW8+SabCE2GvUs651Tqz90MYkeJTkQSBI7kxpZIARGa.E0zQmDppxwmSNu3yN5NGfHJ+59msZR62lz9ksm8cWxQIrgPqR6fPN+76ieedewOaNJVQYIIpXji+qmLhgb9b2tSj5v1gDtDs2NHmuz8.RhlEiSE0ZxHRRBK.43r3KLBb7VBYe98m0hHHRJalHD5XEmx1mGw0yjdzy+QtPzgDvdMOpf1Md9dTkrsRnFC7YQ2pnQD5Px.1gDiZK3h9dRRHx4QtMquJsQv5A0q2by0nj0Xa1mzuesUCZzXsM1r1paRZrwZrpMQN2a2.tVE2USzrDjyRsTAS5FpNSlZfi4I7dBlYPMTWvxoh6nDAFWzHE0NjKBNJOPkffU4nYgsESCaek6A7.9T4yBeOvNAdFhhAPmEtJ8V7JzqVQ5Us.8lCkbJPokRozCc6Ri4izylwvm66tmDxl8IPdpHUR0EsverjaaEngTuRDYHqSLLXJhxMpVsBtd0pKukuuOjrRz31QspgeJlFy.encqUn85P0wSJWhF0yPjjRUvqBnVG9Ta0FUvqYPOEa8aBaW8DgEb8lWCs+ieLtsJpmpk5bruDJORfRE1zEx+BiJdOcNO31VcvuRMVB43YKySm6iOWJ3RFt+XIUyURbNEMFsB97J3IUvmUAGtr+Ed9dBEkHvCAeJOHRBBxMQALfO3kqQByjhFwh0bVBDui9gtu7vTc88tv2yqzY7.cXom.1wLJjwGDpgggU788tb4s.6B+ja7XiiEzt09a2o.MRiL6azXekZ31xfNLlnrEbADqDyFvM616j4ukKEDSNK2CfrQdfn7fJXUueY4TJdJIFS.yARVg.lZqLY5yyDpYmqyEJ5IxjxC.CNRXprLo7RlTqG7.+AxvsdA78.S7IsmP4zuRV4LtNbawnPRY6hnYQoyTqBt5JMW1DbAb8glLYY4WwnZhbffU9DnNB9kbR82BUVq+VChZV.9oF8XhXLKinTSgs0FgpSgNfu+8VVSEb5PVvx3mM00RsO9IVA8FjNbqOh9TqgvviUZGH0Ttz1wbh.hqkqUOm3lH91B9.IK30vZWVCULmPNoZJYaheDP7otfQZIJjhiYAP7yr591.Xp6rOoGSf+V7bWXHQLckquwMsjdoAmKMry2ang5WqNpPITpZwL83XIdnYvk1OoOS2GojGpzrWJKCabryi+3o52etyYpniUBAKdtSaNRI9eBXY43ndr3J3SMY5oJBsMuZuX2atWbwiJnoavJnnRtmjqe4Hl7lN.AksqD5auPFq.U01t1OHqqc9VNDGZO66l2LEYYcwCcQ2pU39tSaoNmk3m1aGhlXNFIiYyZHAx1gcJbHd5gJdt6vRFpUirTOZjRZ7CG+aG4OGBqNeCXyIlW5.uDQrxbgWsc4.KbueM6tCoM5LR9tLIls5.i9Z22.4nX+cIw5P+2vkA9c3wL+2nTA9n7seFf+E7jCblzO7geyJEcK8fhAOKc+y++3BW+be3FGpfwBh9pWCwbYsrIf8CW4rey46xDtdRwhh+ytaxskhOz8HtlFNeNtvb3nY6ym.Nlcituvc298gyMlQvkb67yeZt9FBNmRykCNfni4lpoCGG0ExzTFXcojIL8ibVvTwlNtpYrIBzkICrCLkIYSVyL1IaxZ4SB0ozX06nocKL2Y7yrR.NIs2u1CtnOLFW6iaP.k3vUXeGkd0k5Z.qeWAt5cEXi6Jvl2UfqcWAt9cE3F+6.M+GFaOVqhR21fPGbzt1F5NN6JIPEnsZE824A21d.

[JulesV]

@d-healey Clever! Thank you!