NullReferenceException with new Runtime 2.22.1.0

[hpetriffer]

With the new runtime the following code fails:

public class Test
	public constructor(resource as usual, notUsed := nil as usual, width := 0 as int, height := 0 as int, svgColorMode := ColorMode.Default as ColorMode)
		return
end class

function SafeCreateInstance(symObject as usual, uParams params usual[]) as object
	return CreateInstance(symObject, uParams)


function Start() as void strict
	var x := SafeCreateInstance(#Test, 16, 16)
	Console.ReadLine()

	return


public enum ColorMode

    /// <summary>
    /// Default mode.
    /// </summary>
    Default

    /// <summary>
    /// Light mode.
    /// </summary>
    Light

    /// <summary>
    /// Dark mode.
    /// </summary>
    Dark

    /// <summary>
    /// Custom mode (has to be handled by the user-code).
    /// </summary>
    Custom

end enum

In CreateInstance the following exception is thrown:

XSharp.Error
  HResult=0x80131500
  Message=Der Objektverweis wurde nicht auf eine Objektinstanz festgelegt.
  Source=XSharp.RT
  StackTrace:
 OOPHELPERS:VALUECONVERT (Line: 0)
 OOPHELPERS:GETDEFAULTVALUE (Line: 0)
 OOPHELPERS:MATCHPARAMETERS (Line: 0)
 _CREATEINSTANCE (Line: 0)
 _CREATEINSTANCE (Line: 0)
 CREATEINSTANCE (Line: 0)
 SAFECREATEINSTANCE (Line: 14)
 START (Line: 18)


  This exception was originally thrown at this call stack:
    XSharp.RT.OOPHelpers.ValueConvert(XSharp.__Usual, System.Type)
    XSharp.RT.OOPHelpers.GetDefaultValue(System.Reflection.ParameterInfo)
    XSharp.RT.Functions._CreateInstance(System.Type, XSharp.__Usual[])

Inner Exception 1:
NullReferenceException: Der Objektverweis wurde nicht auf eine Objektinstanz festgelegt.

XSharpTests.zip

[cpyrgas]

Looks like the problem is with enums only, no matter if default values are used or not. Following throws a runtime error in all late bound calls (except for the access):

CLASS TestClass
	EXPORT IVar AS StringSplitOptions
	PROPERTY Prop AS StringSplitOptions AUTO
	METHOD TestMethod(e AS StringSplitOptions) AS INT
	RETURN e
	METHOD TestMethodDefault(n := 1 AS INT, e AS StringSplitOptions) AS INT
	RETURN n
	METHOD TestMethodDefault2(n AS INT, e := StringSplitOptions.RemoveEmptyEntries AS StringSplitOptions) AS INT
	RETURN n
END CLASS
FUNCTION Start() AS VOID
//	LOCAL u AS TestClass
	LOCAL u AS USUAL
	u := TestClass{}
	u:TestMethod(StringSplitOptions.RemoveEmptyEntries)
	u:TestMethodDefault(2,StringSplitOptions.RemoveEmptyEntries)
	u:TestMethodDefault(,StringSplitOptions.RemoveEmptyEntries)

	u:TestMethodDefault2(1,StringSplitOptions.RemoveEmptyEntries)
	u:TestMethodDefault2(2)

	? u:IVar
	u:IVar := StringSplitOptions.None
	? u:Prop
	u:Prop := StringSplitOptions.None

[cpyrgas]

There's no error now, but the original code does not work as expected. See below, when using SafeCreateInstance(), then the parameters are passed all of them as an array to the first parameter of the constructor.

The question though is, SHOULD it work as expected by @hpetriffer ?

PUBLIC CLASS Test
	PUBLIC CONSTRUCTOR(a AS USUAL, b := NIL AS USUAL, c := 0 AS INT, d := 0 AS INT, e := StringSplitOptions.RemoveEmptyEntries AS StringSplitOptions)
		? a,b,c,d,e:ToString()
END CLASS

FUNCTION SafeCreateInstance(symObject AS USUAL, uParams PARAMS USUAL[]) AS OBJECT
	RETURN CreateInstance(symObject, uParams)

FUNCTION Start() AS VOID STRICT
	// CLASS __USUAL[] , nil , 0 , 0 , RemoveEmptyEntries
	SafeCreateInstance(#Test, 16, 16)

	// 16 ,16 , 0 , 0 , RemoveEmptyEntries
	CreateInstance(#Test, 16, 16)

[RobertvanderHulst]

Yes, the parameters are passed as array. The defaults are handled at runtime by the code that prepares for late bound code.

[cpyrgas]

Yes, but it does not work as Hansjoerg expected it. @hpetriffer , did this code work for you before?

[RobertvanderHulst]

The version of create instance that accepts an array of usuals is called _CreateInstance, so I think the code should be:

FUNCTION SafeCreateInstance(symObject AS USUAL, uParams PARAMS USUAL[]) AS OBJECT
	RETURN _CreateInstance(symObject, uParams)