Type safety!

[d.healey]

I'm finding since this change I am getting some fun errors, but the line numbers given are not correct (or at least are not clear).

For example:

[Christoph Hart]

@d-healey ah good catch, it mistakenly interpreted a function as a JSON object (which is technically correct, but it should identify as Function before). Should be fixed now.

[d.healey]

@Christoph-Hart Thank you

[Dan Korneff]

This safety check is pretty aggressive. I'm trying to track down some API call with undefined parameter 0 errors, but HISE isn't being helpful at the moment.
I get an error at a line that doesn't contain code, and if I comment out the ENTIRE script I still get the error
The only way to make it clear the error is to restart HISE.

It seems to be circling around addComboBox, and that combo box initializes with a value of 0.0.

edit:
it seems to initialize to 0.0 if "saveInPreset": "0" . Is that by design?

[d.healey]

@Dan-Korneff said in Type safety!:

I get an error at a line that doesn't contain code

I'm seeing this a lot too. However if you look above the Console, on the same line as the Compile button you might get a more helpful message.

ComboBox values start at 1, so that might be the issue.

[Christoph Hart]

You guys know you can just change a flag in HISE to restore the old compiler strictness?

HISE/hi_scripting/hi_scripting.h at d91fb47d5cec35358dbf56235fb412bc931a723e · christophhart/HISE

The open source framework for sample based instruments - HISE/hi_scripting/hi_scripting.h at d91fb47d5cec35358dbf56235fb412bc931a723e · christophhart/HISE

GitHub (github.com)

I really stand by this decision from a language design perspective, undefined as function arguments cause more pain than it is useful, but I understand how annoying it is if you just want to pull the latest changes and find your interface completely broken because of a new restriction.

Anyways, I've changed the default value here so that old projects load without issues, and you even can deactivate the warning in the Hise preferences (Development -> Warn If Undefined Parameters) if you want to go full on ostrich on your code quality :)

[Dan Korneff]

@Christoph-Hart nah, I'd rather know that these issues exist and deal with them. Thankful for the feature, annoyed with debugging

[Adam_G]

@Christoph-Hart do we just need to change #define HISE_WARN_UNDEFINED_PARAMETER_CALLS 0 to 1? or is there something else that needs to be changed?

[Christoph Hart]

@Adam_G pull the latest changes, then this flag is set and prints the warning. If you then want to disable the warning, untick the box in the preferences.

[Adam_G]

@Christoph-Hart getting some errors on compiling

[d.healey]

@Adam_G I think that means you are low on RAM. Try closing other programs that you are not using when compiling or reduce the number of threads that VS is using during the compilation.

[Lindon]

@d-healey said in Type safety!:

@Adam_G I think that means you are low on RAM. Try closing other programs that you are not using when compiling or reduce the number of threads that VS is using during the compilation.

• its more likely to mean that the plugin is using too many png's or wav files stored inside the plugin itself - past about 50Mb (if I recall correctly) the compiler runs out of heap space....on Windows only

[Adam_G]

@d-healey doh! you were right. too many other programs open

[Lindon]

@Adam_G said in Type safety!:

@d-healey doh! you were right. too many other programs open

careful --- re-running the compile when th eissue is the one I pointed at can sometimes run successfully...

[Dan Korneff]

I'm running into some undefined parameter warnings that I could use a hand with.
I have a function that adds a childPanel to an existing panel. The paint routine is triggering Warning: undefined parameter 0 if the panel hasn't been created yet, or if the childPanel has been removed.

	inline function addButton(panel)
	{
	    local button = panel.addChildPanel();
	    button.set("allowCallbacks", "Clicks & Hover");
	    
	    button.setPaintRoutine(function(g)
	    {
	        g.setColour(this.data.hover ? Colours.red : Colours.blue);
		g.fillPath(myPath, [10, 5, 14, 14]);

	    });

	}

Is there a better way to do this that won't throw these warnings?

[d.healey]

@Dan-Korneff

	inline function addButton(panel)
	{
            if (!isDefined(panel))
                return;

	    local button = panel.addChildPanel();
	    button.set("allowCallbacks", "Clicks & Hover");
	    
	    button.setPaintRoutine(function(g)
	    {
	        g.setColour(this.data.hover ? Colours.red : Colours.blue);
		g.fillPath(myPath, [10, 5, 14, 14]);

	    });

	}

[Dan Korneff]

@d-healey Perfect! Simple and effective.
Actually, this method still returns errors

[d.healey]

@Dan-Korneff

Actually, this method still returns errors

Explain

[Dan Korneff]

@d-healey I've added this to the childPanel functions:

            if (!isDefined(panel))
                return;

• Run the function that creates the child panel.
• Close the panel.
• Hit F5 to recompile All Scripts.

Then I get Warning: undefined parameter 0

After closer inspection, the warning is pointing to the line that contains fillPath. I can confirm that this path is being defined. It points to all paint routines

[Dan Korneff]

Here's a basic example:

HiseSnippet 1334.3ocsW0uaaaCDWJIJs1acXEXO.b9OFjA7BrrcbhSPQasSxZ1VRLlSKJvPP.iDsMQjIMjnchQQd+1iydC1tizxRx4i1ZrI.KHd2u63cGu634tQReVbrLxxtv4yFyrr+VmdyDpgcFR4BqiOvx96bNgFqXQDCo1yFSiiYAV11q+KHA6BaXoe96W2lFRE9rTRVVePx8Y+NeDWkRs6a9MdX3Qz.147QYP23MG6KEcjgxIf8rtSUqwT+qoCXmRQXq4XYu4gAbkLpmhpXw.l1xfY8FJuQXv+AdL+pPFtvypGnHCYqNC4gAcS70XKK6M5l54qa77ev4Dd.eA8zHv2qYPRkHaLvdsmxj79JLI6LlzFFS5kN87i3iUobP64abNV.GH8oPnNqoXvZs1O+LmNR.gPs0H50rihfEKjvsY0pUHvqx6WrHDtiUjozHRWpfE5QdEIQRZPfllaICqRUHfbnTIHhYncMlEo3rXXSF8q8N6zLv+TwBktgGnFVZuZ3dBKGx3CFpJsGIg.MLTdSG38UvIcbo8J0IjCeP9Ix6jSYQkJdWdy73Nmc5kiopgYrT+HFD56BDcAvKgMd5fKCnJJf+O87fM0qZKsm3UaW3USLVrCRE94Ab71YmDJH+Z0qP1tFR1aYf6lEHPsUSTeMLDqivZgba0HCrZ.4FHtsqanVqJvdGjxtMxqNTAMpMm1NMPM0ZY04UGTSi53GaaHuvwV15pUCUQikgk2aeb00zfqHAddTKrdqbQTzeqCtViVYbXse5ssduygzCPp8vjP3inxcwSjlYOQvyHOuloVoQOU8LV4E6WbQVyVgRZ.lpd.jR3lKAAyy3hPtfQ5OQ3q3RAApAZOQojB2wXRc4hPJMuOw8G4wGv5CPClynbwBXjIholDI.EgKBk9zPxUZ4grOMPrpxT3qKsf8DQZXgFwVALJr0Sg741JTJi3FX5WFB5ZOYLGsRWSVMFCvy4FyUZJP2kqxpPVtLq7CuCPCP0eHmn.O0MIn3NnrF1mzuwmAHVSWaW0PdrwSFhJl7ZhgQ7VQr.xdKVMHhMC1zBF46CWGnqeWbPUApVAuYaLEC+cQB3+K10bp3HnKhaoytc1.lfzVFFTB2ubnBhn271P9.379b1s.51u+7yO6zRnQV2TGfsyzVw.l5c5lbtku.hy9PCJvFRBv28vA5SjShYImOoQZ1TP5ki1K6quhngYVkZ0K9f22nms7wibVP4Ebfj4BEflnwxP1Vii3XbvjtS5XvVJSX3tkUr1PhXiwjD2r9Ww6JVLsxwbcP48WTTIEmJUryDt5xI.LYYV86+f7v98QxvPVzCxFmkH5oDzULYzUrnJvkCgSXK.BW1l+F7M+xtA22b8SFfRwwBt5rwr4qOBxlvalwuu+88Vyu+B958Gi8ivQ.lSK8dUf1ArovLTlABJ3b.K9ZkbrF6nwRApA6mozbeQx3BXL2hCa8ycLweqasrcrwgrlk7g9pYPia9WyGUyb4bVJ46afCBkuuALJ28FYAFVRFLIjpxOAENp3bFvgRtwVvQSDPurYYGk7qXrppO4XUeol3Kc5xU9CeXabsGvFgiq+Orw4Ci9BmC62m4qRMvMbN5iq5jmels2zievITUDGSTNcxndPCSeFr6BH8IFyYVCSYMqqljC0iIBzK9G3YNSObs8bldILsFQ8ijW5aJEwwcetlBXSB8z8Ef+lArl3YoKOyFmGASeeouedUcOAqspBVeUErwpJ31qpfMWUA2YUEb2Ouf3eN5sSTxQlxFKqS5dntels8gBJjApyVs9W.ObaiA

Load it up and hit F5