duplicate instance variable definitions are not detected

[telephon]

The following class definition won't fail although it should:

Test {
    var variable;
    var variable;

}

[patrickdupuis]

Tried this on 3.7.1 and got the following error:

ERROR: Class not defined.
  in file 'selected text'
  line 1 char 4:

  Test {

    var variable;
-----------------------------------
ERROR: Function variable 'variable' already declared in Interpreter:functionCompileContext
  in file 'selected text'
  line 3 char 14:

    var variable;

  } 
-----------------------------------

[jamshark70]

Reopening. @patrickdupuis 's test was not valid. The code block in the issue description is a class definition, so the proper test is not to submit it interactively to the interpreter. You have to put it in a .sc file in your extension directory and recompile the library.

I did that test and Julian is correct -- the compiler reports no error.

Not fixed yet.

[mossheim]

Will look into this. Further test:

// TestClass.sc
TestClass {
  var <var1 = 2;
  var <var1 = 3;
}

// Interpreted code
TestClass.new.var1.postln; // prints '3'

There's a missing check somewhere.