Primary constructor & interface static constructor

[dimitrakokkali]

Adds support for:

• Primary constructors
• Interface static constructors

[RobertvanderHulst]

Nikos,
It looks like this has broken our compiler tests.
I have the impression that somehow 2 parameterless constructors are generated because getParameters() returns an empty parameterlist when there are no parameters.
In this case it should return null.
Maybe we need to add a parameter to getParameters() to tell it when to return an Empty parameter list and when to return null?

[nvkokkalis]

Yeah, makes sense!

[nvkokkalis]

I'll try reverting to check the workflow too

[RobertvanderHulst]

And maybe you can also add some tests to the test suite to make sure that this works.
I would recommend:

• class without primary constructor and clipper constructor
• class with primary constructor and clipper constructor (this should fail), has to go to a different test app
• class with just a primary constructor
• class with primary constructor and one or more typed constructors
• and the same of course for structures
• and the same for record class and record structure

[nvkokkalis]

I was wondering about that... how are the tests organised?

We have Cxxxx and Rxxxx tests in different folders. Is there some structure there or do we just add a new file?

Also, primary constructors in some cases (in records) require dotnet SDK to be supported. Do we support SDK projects in the tests?

[RobertvanderHulst]

The tests that start with 'C' are created by Chris. The ones that start with 'R' by me. I suggest you create tests that start with 'N' or 'D'. And you can create a project that targets .Net 8 or later with XIDE as well.
We try to reference the runtime DLLs from the src\CompilerTests\Runtime folder, so they do not have to be in the GAC.
I suggest you create them and put them in New tree node.
If the tests are ready then we move them to either the Fixed tree node or the "Must Report Error" tree node.
After Chris or I validate them we put them in the right tree node.
To run the tests (after successful compilation) in the suite you reference them in the RuntimeTests.prg . I suspect you cannot do that for .Net 8 tests because that app is .Net Framework.
But we can of course also create a sibling app to run .Net 8 tests. We would have to adjust the batch file that runs the tests to compile and run 2 test-runners.

[cpyrgas]

Also, Nik, in order to run the tests from within XIDE, you need to right click on the "FIXED" node in the Project pad, and select "Rebuild all contained apps" from the context menu. Please make sure you have the XIDE 3 version installed, as it has some project file changes compared to XIDE for X# 2.