JUCE icon indicating copy to clipboard operation
JUCE copied to clipboard

[Bug]: (JUCE8) variant.isVoid() not handled in VariantConverter fromVar()

Open benkuper opened this issue 1 year ago • 0 comments

Detailed steps on how to reproduce the bug

Using the new QuickJS JS Engine, calling from a script a C function that returns a var() (variant Void) leads to an assert because this is not a handled case in the function :

struct VariantConverter<choc::value::Value>
{
    static choc::value::Value fromVar (const var& variant)

Or is var() as return type not allowed / good practice ? It has been working great with the [old] engine

What is the expected behaviour?

The end of the function

        if (variant.isUndefined())
            return {};

could be

        if (variant.isUndefined() || variant.isVoid())
            return {};

Operating systems

Windows

What versions of the operating systems?

11

Architectures

64-bit

Stacktrace

No response

Plug-in formats (if applicable)

No response

Plug-in host applications (DAWs) (if applicable)

No response

Testing on the develop branch

The bug is present on the develop branch

Code of Conduct

  • [X] I agree to follow the Code of Conduct

benkuper avatar Jun 23 '24 08:06 benkuper