Browse Source

* error when properties are passed to var parameters

peter 26 years ago
parent
commit
8e7dd52200
8 changed files with 116 additions and 87 deletions
  1. 2 2
      compiler/errore.msg
  2. 1 1
      compiler/msgidx.inc
  3. 69 69
      compiler/msgtxt.inc
  4. 8 1
      compiler/pexpr.pas
  5. 4 2
      compiler/ptype.pas
  6. 8 1
      compiler/tccal.pas
  7. 19 10
      compiler/tcinl.pas
  8. 5 1
      compiler/tree.pas

+ 2 - 2
compiler/errore.msg

@@ -915,11 +915,11 @@ type_e_wrong_math_argument=E_Illegal constant passed to internal math function
 type_e_no_addr_of_constant=E_Can't get the address of constants
 type_e_no_addr_of_constant=E_Can't get the address of constants
 % It's not possible to get the address of a constant, because they
 % It's not possible to get the address of a constant, because they
 % aren't stored in memory, you can try making it a typed constant.
 % aren't stored in memory, you can try making it a typed constant.
-type_e_argument_must_be_lvalue=E_Argument must be l-value
+type_e_argument_cant_be_assigned=E_Argument can't be assigned to
 % Only expressions which can be on the left side of an
 % Only expressions which can be on the left side of an
 % assignment can be passed as call by reference argument
 % assignment can be passed as call by reference argument
 % Remark: Properties can be only
 % Remark: Properties can be only
-% used on the left side of an assignment, but they can't be used as l-values
+% used on the left side of an assignment, but they can't be used as arguments
 type_e_cannot_local_proc_to_procvar=E_Can't assign local procedure/function to procedure variable
 type_e_cannot_local_proc_to_procvar=E_Can't assign local procedure/function to procedure variable
 % It's not allowed to assign a local procedure/function to a
 % It's not allowed to assign a local procedure/function to a
 % procedure variable, because the calling of local procedure/function is
 % procedure variable, because the calling of local procedure/function is

+ 1 - 1
compiler/msgidx.inc

@@ -240,7 +240,7 @@ type tmsgconst=(
   type_e_no_method_and_procedure_not_compatible,
   type_e_no_method_and_procedure_not_compatible,
   type_e_wrong_math_argument,
   type_e_wrong_math_argument,
   type_e_no_addr_of_constant,
   type_e_no_addr_of_constant,
-  type_e_argument_must_be_lvalue,
+  type_e_argument_cant_be_assigned,
   type_e_cannot_local_proc_to_procvar,
   type_e_cannot_local_proc_to_procvar,
   type_e_no_assign_to_addr,
   type_e_no_assign_to_addr,
   sym_e_id_not_found,
   sym_e_id_not_found,

+ 69 - 69
compiler/msgtxt.inc

@@ -255,24 +255,24 @@ const msgtxt : array[0..000103,1..240] of char=(
   'E_Method (variable) and Procedure (variable) are not compatible'#000+
   'E_Method (variable) and Procedure (variable) are not compatible'#000+
   'E_Ille','gal constant passed to internal math function'#000+
   'E_Ille','gal constant passed to internal math function'#000+
   'E_Can'#039't get the address of constants'#000+
   'E_Can'#039't get the address of constants'#000+
-  'E_Argument must be l-value'#000+
+  'E_Argument can'#039't be assigned to'#000+
   'E_Can'#039't assign local procedure/function to procedure variable'#000+
   'E_Can'#039't assign local procedure/function to procedure variable'#000+
   'E_Can'#039't assign values to an address'#000+
   'E_Can'#039't assign values to an address'#000+
   'E_Identifier not found $1'#000+
   'E_Identifier not found $1'#000+
-  'F_Inte','rnal Error in SymTableStack()'#000+
+  'F','_Internal Error in SymTableStack()'#000+
   'E_Duplicate identifier $1'#000+
   'E_Duplicate identifier $1'#000+
   'H_Identifier already defined in $1 at line $2'#000+
   'H_Identifier already defined in $1 at line $2'#000+
   'E_Unknown identifier $1'#000+
   'E_Unknown identifier $1'#000+
   'E_Forward declaration not solved $1'#000+
   'E_Forward declaration not solved $1'#000+
   'F_Identifier type already defined as type'#000+
   'F_Identifier type already defined as type'#000+
   'E_Error in type definition'#000+
   'E_Error in type definition'#000+
-  'E_Type id','entifier not defined'#000+
+  'E_Ty','pe identifier not defined'#000+
   'E_Forward type not resolved $1'#000+
   'E_Forward type not resolved $1'#000+
   'E_Only static variables can be used in static methods or outside metho'+
   'E_Only static variables can be used in static methods or outside metho'+
   'ds'#000+
   'ds'#000+
   'E_Invalid call to tvarsym.mangledname()'#000+
   'E_Invalid call to tvarsym.mangledname()'#000+
   'F_record or class type expected'#000+
   'F_record or class type expected'#000+
-  'E_Instances of classes or objects with an a','bstract method are not al'+
+  'E_Instances of classes or objects with',' an abstract method are not al'+
   'lowed'#000+
   'lowed'#000+
   'W_Label not defined $1'#000+
   'W_Label not defined $1'#000+
   'E_Illegal label declaration'#000+
   'E_Illegal label declaration'#000+
@@ -280,21 +280,21 @@ const msgtxt : array[0..000103,1..240] of char=(
   'E_Label not found'#000+
   'E_Label not found'#000+
   'E_identifier isn'#039't a label'#000+
   'E_identifier isn'#039't a label'#000+
   'E_label already defined'#000+
   'E_label already defined'#000+
-  'E_illegal type declaration of set ele','ments'#000+
+  'E_illegal type declaration of se','t elements'#000+
   'E_Forward class definition not resolved $1'#000+
   'E_Forward class definition not resolved $1'#000+
   'H_Parameter not used $1'#000+
   'H_Parameter not used $1'#000+
   'N_Local variable not used $1'#000+
   'N_Local variable not used $1'#000+
   'E_Set type expected'#000+
   'E_Set type expected'#000+
   'W_Function result does not seem to be set'#000+
   'W_Function result does not seem to be set'#000+
   'E_Unknown record field identifier $1'#000+
   'E_Unknown record field identifier $1'#000+
-  'W_Local variable $1 does not seem to be',' initialized'#000+
+  'W_Local variable $1 does not seem ','to be initialized'#000+
   'W_Variable $1 does not seem to be initialized'#000+
   'W_Variable $1 does not seem to be initialized'#000+
   'E_identifier idents no member $1'#000+
   'E_identifier idents no member $1'#000+
   'B_Found declaration: $1'#000+
   'B_Found declaration: $1'#000+
   'E_Data segment too large (max. 2GB)'#000+
   'E_Data segment too large (max. 2GB)'#000+
   'E_BREAK not allowed'#000+
   'E_BREAK not allowed'#000+
   'E_CONTINUE not allowed'#000+
   'E_CONTINUE not allowed'#000+
-  'E_Expression too complicated - FPU stack over','flow'#000+
+  'E_Expression too complicated - FPU stack',' overflow'#000+
   'E_Illegal expression'#000+
   'E_Illegal expression'#000+
   'E_Invalid integer expression'#000+
   'E_Invalid integer expression'#000+
   'E_Illegal qualifier'#000+
   'E_Illegal qualifier'#000+
@@ -302,92 +302,92 @@ const msgtxt : array[0..000103,1..240] of char=(
   'E_Illegal counter variable'#000+
   'E_Illegal counter variable'#000+
   'E_Can'#039't determine which overloaded function to call'#000+
   'E_Can'#039't determine which overloaded function to call'#000+
   'E_Parameter list size exceeds 65535 bytes'#000+
   'E_Parameter list size exceeds 65535 bytes'#000+
-  'E_Illeg','al type conversion'#000+
+  'E_','Illegal type conversion'#000+
   'D_Conversion between ordinals and pointers is not portable across plat'+
   'D_Conversion between ordinals and pointers is not portable across plat'+
   'forms'#000+
   'forms'#000+
   'E_File types must be var parameters'#000+
   'E_File types must be var parameters'#000+
   'E_The use of a far pointer isn'#039't allowed there'#000+
   'E_The use of a far pointer isn'#039't allowed there'#000+
   'E_illegal call by reference parameters'#000+
   'E_illegal call by reference parameters'#000+
-  'E_EXPORT declared funct','ions can'#039't be called'#000+
+  'E_EXPORT declared ','functions can'#039't be called'#000+
   'W_Possible illegal call of constructor or destructor (doesn'#039't matc'+
   'W_Possible illegal call of constructor or destructor (doesn'#039't matc'+
   'h to this context)'#000+
   'h to this context)'#000+
   'N_Inefficient code'#000+
   'N_Inefficient code'#000+
   'W_unreachable code'#000+
   'W_unreachable code'#000+
   'E_procedure call with stackframe ESP/SP'#000+
   'E_procedure call with stackframe ESP/SP'#000+
   'E_Abstract methods can'#039't be called directly'#000+
   'E_Abstract methods can'#039't be called directly'#000+
-  'F_Internal E','rror in getfloatreg(), allocation failure'#000+
+  'F_Inter','nal Error in getfloatreg(), allocation failure'#000+
   'F_Unknown float type'#000+
   'F_Unknown float type'#000+
   'F_SecondVecn() base defined twice'#000+
   'F_SecondVecn() base defined twice'#000+
   'F_Extended cg68k not supported'#000+
   'F_Extended cg68k not supported'#000+
   'F_32-bit unsigned not supported in MC68000 mode'#000+
   'F_32-bit unsigned not supported in MC68000 mode'#000+
   'F_Internal Error in secondinline()'#000+
   'F_Internal Error in secondinline()'#000+
-  'D_Register $1 weight $2 $3'#000+
-  'E_','Stack limit excedeed in local routine'#000+
+  'D_Register $1 weight $2 ','$3'#000+
+  'E_Stack limit excedeed in local routine'#000+
   'D_Stack frame is omitted'#000+
   'D_Stack frame is omitted'#000+
   'E_Object or class methods can'#039't be inline.'#000+
   'E_Object or class methods can'#039't be inline.'#000+
   'E_Procvar calls can'#039't be inline.'#000+
   'E_Procvar calls can'#039't be inline.'#000+
   'E_No code for inline procedure stored'#000+
   'E_No code for inline procedure stored'#000+
   'E_Direct call of interrupt procedure $1 is not possible'#000+
   'E_Direct call of interrupt procedure $1 is not possible'#000+
-  'E_Eleme','nt zero of an ansi/wide- or longstring can'#039't be accessed,'+
+  'E_','Element zero of an ansi/wide- or longstring can'#039't be accessed,'+
   ' use (set)length instead'#000+
   ' use (set)length instead'#000+
   'E_Include and exclude not implemented in this case'#000+
   'E_Include and exclude not implemented in this case'#000+
   'E_Constructors or destructors can not be called inside a '#039'with'#039+
   'E_Constructors or destructors can not be called inside a '#039'with'#039+
   ' clause'#000+
   ' clause'#000+
-  'E_Cannot call message handler method',' directly'#000+
+  'E_Cannot call message handler m','ethod directly'#000+
   'D_Starting $1 styled assembler parsing'#000+
   'D_Starting $1 styled assembler parsing'#000+
   'D_Finished $1 styled assembler parsing'#000+
   'D_Finished $1 styled assembler parsing'#000+
   'E_Non-label pattern contains @'#000+
   'E_Non-label pattern contains @'#000+
   'W_Override operator not supported'#000+
   'W_Override operator not supported'#000+
   'E_Error building record offset'#000+
   'E_Error building record offset'#000+
   'E_OFFSET used without identifier'#000+
   'E_OFFSET used without identifier'#000+
-  'E_TYPE used without ide','ntifier'#000+
+  'E_TYPE used withou','t identifier'#000+
   'E_Cannot use local variable or parameters here'#000+
   'E_Cannot use local variable or parameters here'#000+
   'E_need to use OFFSET here'#000+
   'E_need to use OFFSET here'#000+
   'E_Cannot use multiple relocatable symbols'#000+
   'E_Cannot use multiple relocatable symbols'#000+
   'E_Relocatable symbol can only be added'#000+
   'E_Relocatable symbol can only be added'#000+
   'E_Invalid constant expression'#000+
   'E_Invalid constant expression'#000+
   'E_Relocatable symbol is not allowed'#000+
   'E_Relocatable symbol is not allowed'#000+
-  'E_Invalid re','ference syntax'#000+
+  'E_Inval','id reference syntax'#000+
   'E_Local symbols/labels aren'#039't allowed as references'#000+
   'E_Local symbols/labels aren'#039't allowed as references'#000+
   'E_Invalid base and index register usage'#000+
   'E_Invalid base and index register usage'#000+
   'E_Wrong scale factor specified'#000+
   'E_Wrong scale factor specified'#000+
   'E_Multiple index register usage'#000+
   'E_Multiple index register usage'#000+
   'E_Invalid operand type'#000+
   'E_Invalid operand type'#000+
   'E_Invalid string as opcode operand: $1'#000+
   'E_Invalid string as opcode operand: $1'#000+
-  'W_@CODE ','and @DATA not supported'#000+
+  'W_@','CODE and @DATA not supported'#000+
   'E_Null label references are not allowed'#000+
   'E_Null label references are not allowed'#000+
   'F_Divide by zero in asm evaluator'#000+
   'F_Divide by zero in asm evaluator'#000+
   'F_Evaluator stack overflow'#000+
   'F_Evaluator stack overflow'#000+
   'F_Evaluator stack underflow'#000+
   'F_Evaluator stack underflow'#000+
   'F_Invalid numeric format in asm evaluator'#000+
   'F_Invalid numeric format in asm evaluator'#000+
   'F_Invalid Operator in asm evaluator'#000+
   'F_Invalid Operator in asm evaluator'#000+
-  'E_escape ','sequence ignored: $1'#000+
+  'E_es','cape sequence ignored: $1'#000+
   'E_Invalid symbol reference'#000+
   'E_Invalid symbol reference'#000+
   'W_Fwait can cause emulation problems with emu387'#000+
   'W_Fwait can cause emulation problems with emu387'#000+
   'W_Calling an overload function in assembler'#000+
   'W_Calling an overload function in assembler'#000+
   'E_Unsupported symbol type for operand'#000+
   'E_Unsupported symbol type for operand'#000+
   'E_Constant value out of bounds'#000+
   'E_Constant value out of bounds'#000+
-  'E_Error converting decimal $1'#000,
+  'E_Error converting decima','l $1'#000+
   'E_Error converting octal $1'#000+
   'E_Error converting octal $1'#000+
   'E_Error converting binary $1'#000+
   'E_Error converting binary $1'#000+
   'E_Error converting hexadecimal $1'#000+
   'E_Error converting hexadecimal $1'#000+
   'H_$1 translated to $2'#000+
   'H_$1 translated to $2'#000+
   'W_$1 is associated to an overloaded function'#000+
   'W_$1 is associated to an overloaded function'#000+
   'E_Cannot use SELF outside a method'#000+
   'E_Cannot use SELF outside a method'#000+
-  'E_Cannot use OLDEBP outside a nested procedure'#000,
+  'E_Cannot use OLDEBP outside a nested proce','dure'#000+
   'W_Functions with void return value can'#039't return any value in asm c'+
   'W_Functions with void return value can'#039't return any value in asm c'+
   'ode'#000+
   'ode'#000+
   'E_SEG not supported'#000+
   'E_SEG not supported'#000+
   'E_Size suffix and destination or source size do not match'#000+
   'E_Size suffix and destination or source size do not match'#000+
   'W_Size suffix and destination or source size do not match'#000+
   'W_Size suffix and destination or source size do not match'#000+
   'E_Assembler syntax error'#000+
   'E_Assembler syntax error'#000+
-  'E_Invalid',' combination of opcode and operands'#000+
+  'E_In','valid combination of opcode and operands'#000+
   'E_Assemler syntax error in operand'#000+
   'E_Assemler syntax error in operand'#000+
   'E_Assemler syntax error in constant'#000+
   'E_Assemler syntax error in constant'#000+
   'E_Invalid String expression'#000+
   'E_Invalid String expression'#000+
   '32bit constant created for address'#000+
   '32bit constant created for address'#000+
   'E_Invalid or missing opcode'#000+
   'E_Invalid or missing opcode'#000+
-  'E_Invalid combination of prefix and opcode',': $1'#000+
+  'E_Invalid combination of prefix and o','pcode: $1'#000+
   'E_Invalid combination of override and opcode: $1'#000+
   'E_Invalid combination of override and opcode: $1'#000+
   'E_Too many operands on line'#000+
   'E_Too many operands on line'#000+
   'W_NEAR ignored'#000+
   'W_NEAR ignored'#000+
@@ -395,69 +395,69 @@ const msgtxt : array[0..000103,1..240] of char=(
   'E_Duplicate local symbol $1'#000+
   'E_Duplicate local symbol $1'#000+
   'E_Undefined local symbol $1'#000+
   'E_Undefined local symbol $1'#000+
   'E_Unknown label identifier $1'#000+
   'E_Unknown label identifier $1'#000+
-  'E_Invalid floating point register name'#000+
-  'E_NO','R not supported'#000+
+  'E_Invalid floating point register name',#000+
+  'E_NOR not supported'#000+
   'W_Modulo not supported'#000+
   'W_Modulo not supported'#000+
   'E_Invalid floating point constant $1'#000+
   'E_Invalid floating point constant $1'#000+
   'E_Invalid floating point expression'#000+
   'E_Invalid floating point expression'#000+
   'E_Wrong symbol type'#000+
   'E_Wrong symbol type'#000+
   'E_Cannot index a local var or parameter with a register'#000+
   'E_Cannot index a local var or parameter with a register'#000+
   'E_Invalid segment override expression'#000+
   'E_Invalid segment override expression'#000+
-  'W_Identifier $','1 supposed external'#000+
+  'W_Identif','ier $1 supposed external'#000+
   'E_Strings not allowed as constants'#000+
   'E_Strings not allowed as constants'#000+
   'No type of variable specified'#000+
   'No type of variable specified'#000+
   'E_assembler code not returned to text section'#000+
   'E_assembler code not returned to text section'#000+
   'E_Not a directive or local symbol $1'#000+
   'E_Not a directive or local symbol $1'#000+
   'E_Using a defined name as a local label'#000+
   'E_Using a defined name as a local label'#000+
-  'E_Dollar token is used without a','n identifier'#000+
+  'E_Dollar token is used with','out an identifier'#000+
   'W_32bit constant created for address'#000+
   'W_32bit constant created for address'#000+
   'N_.align is target specific, use .balign or .p2align'#000+
   'N_.align is target specific, use .balign or .p2align'#000+
   'E_Can'#039't access fields directly for parameters'#000+
   'E_Can'#039't access fields directly for parameters'#000+
   'E_Can'#039't access fields of objects/classes directly'#000+
   'E_Can'#039't access fields of objects/classes directly'#000+
   'F_Too many assembler files'#000+
   'F_Too many assembler files'#000+
-  'F_Selected ass','embler output not supported'#000+
+  'F_Selecte','d assembler output not supported'#000+
   'F_Comp not supported'#000+
   'F_Comp not supported'#000+
   'F_Direct not support for binary writers'#000+
   'F_Direct not support for binary writers'#000+
   'E_Allocating of data is only allowed in bss section'#000+
   'E_Allocating of data is only allowed in bss section'#000+
   'F_No binary writer selected'#000+
   'F_No binary writer selected'#000+
   'E_Asm: Opcode $1 not in table'#000+
   'E_Asm: Opcode $1 not in table'#000+
-  'E_Asm: $1 invalid combination of opcode a','nd operands'#000+
+  'E_Asm: $1 invalid combination of opc','ode and operands'#000+
   'E_Asm: 16 Bit references not supported'#000+
   'E_Asm: 16 Bit references not supported'#000+
   'E_Asm: Invalid effective address'#000+
   'E_Asm: Invalid effective address'#000+
   'E_Asm: Immediate or reference expected'#000+
   'E_Asm: Immediate or reference expected'#000+
   'E_Asm: $1 value exceeds bounds $2'#000+
   'E_Asm: $1 value exceeds bounds $2'#000+
   'E_Asm: Short jump is out of range $1'#000+
   'E_Asm: Short jump is out of range $1'#000+
   'W_Source operating system redefined'#000+
   'W_Source operating system redefined'#000+
-  'I_Assembli','ng (pipe) $1'#000+
+  'I_Ass','embling (pipe) $1'#000+
   'E_Can'#039't create assember file $1'#000+
   'E_Can'#039't create assember file $1'#000+
   'W_Assembler $1 not found, switching to external assembling'#000+
   'W_Assembler $1 not found, switching to external assembling'#000+
   'T_Using assembler: $1'#000+
   'T_Using assembler: $1'#000+
   'W_Error while assembling exitcode $1'#000+
   'W_Error while assembling exitcode $1'#000+
   'W_Can'#039't call the assembler, error $1 switching to external assembl'+
   'W_Can'#039't call the assembler, error $1 switching to external assembl'+
   'ing'#000+
   'ing'#000+
-  'I_Assem','bling $1'#000+
+  'I_','Assembling $1'#000+
   'I_Assembling smartlink $1'#000+
   'I_Assembling smartlink $1'#000+
   'W_Object $1 not found, Linking may fail !'#000+
   'W_Object $1 not found, Linking may fail !'#000+
   'W_Library $1 not found, Linking may fail !'#000+
   'W_Library $1 not found, Linking may fail !'#000+
   'W_Error while linking'#000+
   'W_Error while linking'#000+
   'W_Can'#039't call the linker, switching to external linking'#000+
   'W_Can'#039't call the linker, switching to external linking'#000+
   'I_Linking $1'#000+
   'I_Linking $1'#000+
-  'W_Util $1 not found, switching',' to external linking'#000+
+  'W_Util $1 not found, swit','ching to external linking'#000+
   'T_Using util $1'#000+
   'T_Using util $1'#000+
   'E_Creation of Executables not supported'#000+
   'E_Creation of Executables not supported'#000+
   'E_Creation of Dynamic/Shared Libraries not supported'#000+
   'E_Creation of Dynamic/Shared Libraries not supported'#000+
   'I_Closing script $1'#000+
   'I_Closing script $1'#000+
   'W_resource compiler not found, switching to external mode'#000+
   'W_resource compiler not found, switching to external mode'#000+
   'I_Compiling resource $1'#000+
   'I_Compiling resource $1'#000+
-  'F_Can'#039't ','post process executable $1'#000+
+  'F_C','an'#039't post process executable $1'#000+
   'F_Can'#039't open executable $1'#000+
   'F_Can'#039't open executable $1'#000+
   'X_Size of Code: $1 bytes'#000+
   'X_Size of Code: $1 bytes'#000+
   'X_Size of initialized data: $1 bytes'#000+
   'X_Size of initialized data: $1 bytes'#000+
   'X_Size of uninitialized data: $1 bytes'#000+
   'X_Size of uninitialized data: $1 bytes'#000+
   'X_Stack space reserved: $1 bytes'#000+
   'X_Stack space reserved: $1 bytes'#000+
   'X_Stack space commited: $1 bytes'#000+
   'X_Stack space commited: $1 bytes'#000+
-  'T_Unitsearch: $1'#000+
-  'T_','PPU Loading $1'#000+
+  'T_Unitsearch: ','$1'#000+
+  'T_PPU Loading $1'#000+
   'U_PPU Name: $1'#000+
   'U_PPU Name: $1'#000+
   'U_PPU Flags: $1'#000+
   'U_PPU Flags: $1'#000+
   'U_PPU Crc: $1'#000+
   'U_PPU Crc: $1'#000+
@@ -466,7 +466,7 @@ const msgtxt : array[0..000103,1..240] of char=(
   'U_PPU Invalid Header (no PPU at the begin)'#000+
   'U_PPU Invalid Header (no PPU at the begin)'#000+
   'U_PPU Invalid Version $1'#000+
   'U_PPU Invalid Version $1'#000+
   'U_PPU is compiled for an other processor'#000+
   'U_PPU is compiled for an other processor'#000+
-  'U_PPU is compiled for an other targ','et'#000+
+  'U_PPU is compiled for an other',' target'#000+
   'U_PPU Source: $1'#000+
   'U_PPU Source: $1'#000+
   'U_Writing $1'#000+
   'U_Writing $1'#000+
   'F_Can'#039't Write PPU-File'#000+
   'F_Can'#039't Write PPU-File'#000+
@@ -476,26 +476,26 @@ const msgtxt : array[0..000103,1..240] of char=(
   'F_PPU Dbx count problem'#000+
   'F_PPU Dbx count problem'#000+
   'E_Illegal unit name: $1'#000+
   'E_Illegal unit name: $1'#000+
   'F_Too much units'#000+
   'F_Too much units'#000+
-  'F_Circular unit reference between $1 and $','2'#000+
+  'F_Circular unit reference between $1 ','and $2'#000+
   'F_Can'#039't compile unit $1, no sources available'#000+
   'F_Can'#039't compile unit $1, no sources available'#000+
   'F_Can'#039't find unit $1'#000+
   'F_Can'#039't find unit $1'#000+
   'W_Unit $1 was not found but $2 exists'#000+
   'W_Unit $1 was not found but $2 exists'#000+
   'F_Unit $1 searched but $2 found'#000+
   'F_Unit $1 searched but $2 found'#000+
   'W_Compiling the system unit requires the -Us switch'#000+
   'W_Compiling the system unit requires the -Us switch'#000+
-  'F_There were $1 errors compiling module, stopping',#000+
+  'F_There were $1 errors compiling module, sto','pping'#000+
   'U_Load from $1 ($2) unit $3'#000+
   'U_Load from $1 ($2) unit $3'#000+
   'U_Recompiling $1, checksum changed for $2'#000+
   'U_Recompiling $1, checksum changed for $2'#000+
   'U_Recompiling $1, source found only'#000+
   'U_Recompiling $1, source found only'#000+
   'U_Recompiling unit, static lib is older than ppufile'#000+
   'U_Recompiling unit, static lib is older than ppufile'#000+
   'U_Recompiling unit, shared lib is older than ppufile'#000+
   'U_Recompiling unit, shared lib is older than ppufile'#000+
-  'U_Recompiling unit, obj and',' asm are older than ppufile'#000+
+  'U_Recompiling unit, ob','j and asm are older than ppufile'#000+
   'U_Recompiling unit, obj is older than asm'#000+
   'U_Recompiling unit, obj is older than asm'#000+
   'U_Parsing interface of $1'#000+
   'U_Parsing interface of $1'#000+
   'U_Parsing implementation of $1'#000+
   'U_Parsing implementation of $1'#000+
   'U_Second load for unit $1'#000+
   'U_Second load for unit $1'#000+
   'U_PPU Check file $1 time $2'#000+
   'U_PPU Check file $1 time $2'#000+
   '$1 [options] <inputfile> [options]'#000+
   '$1 [options] <inputfile> [options]'#000+
-  'W_Only one source file s','upported'#000+
+  'W_Only one source f','ile supported'#000+
   'W_DEF file can be created only for OS/2'#000+
   'W_DEF file can be created only for OS/2'#000+
   'E_nested response files are not supported'#000+
   'E_nested response files are not supported'#000+
   'F_No source file name in command line'#000+
   'F_No source file name in command line'#000+
@@ -503,18 +503,18 @@ const msgtxt : array[0..000103,1..240] of char=(
   'H_-? writes help pages'#000+
   'H_-? writes help pages'#000+
   'F_Too many config files nested'#000+
   'F_Too many config files nested'#000+
   'F_Unable to open file $1'#000+
   'F_Unable to open file $1'#000+
-  'N_Readin','g further options from $1'#000+
+  'N_R','eading further options from $1'#000+
   'W_Target is already set to: $1'#000+
   'W_Target is already set to: $1'#000+
   'W_Shared libs not supported on DOS platform, reverting to static'#000+
   'W_Shared libs not supported on DOS platform, reverting to static'#000+
   'F_too many IF(N)DEFs'#000+
   'F_too many IF(N)DEFs'#000+
   'F_too many ENDIFs'#000+
   'F_too many ENDIFs'#000+
   'F_open conditional at the end of the file'#000+
   'F_open conditional at the end of the file'#000+
-  'W_Debug information generation is not',' supported by this executable'#000+
+  'W_Debug information generation i','s not supported by this executable'#000+
   'H_Try recompiling with -dGDB'#000+
   'H_Try recompiling with -dGDB'#000+
   'E_You are using the obsolete switch $1'#000+
   'E_You are using the obsolete switch $1'#000+
   'E_You are using the obsolete switch $1, please use $2'#000+
   'E_You are using the obsolete switch $1, please use $2'#000+
   'N_Switching assembler to default source writing assembler'#000+
   'N_Switching assembler to default source writing assembler'#000+
-  'Free Pascal Compiler version $','FPCVER [$FPCDATE] for $FPCTARGET'#000+
+  'Free Pascal Compiler vers','ion $FPCVER [$FPCDATE] for $FPCTARGET'#000+
   'Copyright (c) 1993-1999 by Florian Klaempfl'#000+
   'Copyright (c) 1993-1999 by Florian Klaempfl'#000+
   'Free Pascal Compiler version $FPCVER'#000+
   'Free Pascal Compiler version $FPCVER'#000+
   #000+
   #000+
@@ -522,19 +522,19 @@ const msgtxt : array[0..000103,1..240] of char=(
   'Compiler Target: $FPCTARGET'#000+
   'Compiler Target: $FPCTARGET'#000+
   #000+
   #000+
   'This program comes under the GNU General Public Licence'#000+
   'This program comes under the GNU General Public Licence'#000+
-  'For more infor','mation read COPYING.FPC'#000+
+  'For more ','information read COPYING.FPC'#000+
   #000+
   #000+
   'Report bugs,suggestions etc to:'#000+
   'Report bugs,suggestions etc to:'#000+
   '                 [email protected]'#000+
   '                 [email protected]'#000+
   '**0*_put + after a boolean switch option to enable it, - to disable it'+
   '**0*_put + after a boolean switch option to enable it, - to disable it'+
   #000+
   #000+
-  '**1a_the compiler doesn'#039't delete the generated assembler file',#000+
+  '**1a_the compiler doesn'#039't delete the generated assembler',' file'#000+
   '**2al_list sourcecode lines in assembler file'#000+
   '**2al_list sourcecode lines in assembler file'#000+
   '**2ar_list register allocation/release info in assembler file'#000+
   '**2ar_list register allocation/release info in assembler file'#000+
   '**2at_list temp allocation/release info in assembler file'#000+
   '**2at_list temp allocation/release info in assembler file'#000+
   '**1b_generate browser info'#000+
   '**1b_generate browser info'#000+
   '**2bl_generate local symbol info'#000+
   '**2bl_generate local symbol info'#000+
-  '**1B_build al','l modules'#000+
+  '**1B_bui','ld all modules'#000+
   '**1C<x>_code generation options:'#000+
   '**1C<x>_code generation options:'#000+
   '3*2CD_create dynamic library'#000+
   '3*2CD_create dynamic library'#000+
   '**2Ch<n>_<n> bytes heap (between 1023 and 67107840)'#000+
   '**2Ch<n>_<n> bytes heap (between 1023 and 67107840)'#000+
@@ -542,27 +542,27 @@ const msgtxt : array[0..000103,1..240] of char=(
   '**2Cn_omit linking stage'#000+
   '**2Cn_omit linking stage'#000+
   '**2Co_check overflow of integer operations'#000+
   '**2Co_check overflow of integer operations'#000+
   '**2Cr_range checking'#000+
   '**2Cr_range checking'#000+
-  '**2Cs<n>_','set stack size to <n>'#000+
+  '**2C','s<n>_set stack size to <n>'#000+
   '**2Ct_stack checking'#000+
   '**2Ct_stack checking'#000+
   '**2CD_create also dynamic library (* doesn'#039't work yet *)'#000+
   '**2CD_create also dynamic library (* doesn'#039't work yet *)'#000+
   '**2CX_create also smartlinked library'#000+
   '**2CX_create also smartlinked library'#000+
   '**1d<x>_defines the symbol <x>'#000+
   '**1d<x>_defines the symbol <x>'#000+
   '*O1D_generate a DEF file'#000+
   '*O1D_generate a DEF file'#000+
   '*O2Dd<x>_set description to <x>'#000+
   '*O2Dd<x>_set description to <x>'#000+
-  '*O2Dw_PM appli','cation'#000+
+  '*O2Dw_PM ','application'#000+
   '**1e<x>_set path to executable'#000+
   '**1e<x>_set path to executable'#000+
   '**1E_same as -Cn'#000+
   '**1E_same as -Cn'#000+
   '**1F<x>_set file names and paths:'#000+
   '**1F<x>_set file names and paths:'#000+
   '**2FD<x>_sets the directory where to search for compiler utilities'#000+
   '**2FD<x>_sets the directory where to search for compiler utilities'#000+
   '**2Fe<x>_redirect error output to <x>'#000+
   '**2Fe<x>_redirect error output to <x>'#000+
-  '**2FE<x>_set exe/unit output path to <x>'#000+
-  '**2Fi','<x>_adds <x> to include path'#000+
+  '**2FE<x>_set exe/unit output path to <x>'#000,
+  '**2Fi<x>_adds <x> to include path'#000+
   '**2Fl<x>_adds <x> to library path'#000+
   '**2Fl<x>_adds <x> to library path'#000+
   '*L2FL<x>_uses <x> as dynamic linker'#000+
   '*L2FL<x>_uses <x> as dynamic linker'#000+
   '**2Fo<x>_adds <x> to object path'#000+
   '**2Fo<x>_adds <x> to object path'#000+
   '**2Fr<x>_load error message file <x>'#000+
   '**2Fr<x>_load error message file <x>'#000+
   '**2Fu<x>_adds <x> to unit path'#000+
   '**2Fu<x>_adds <x> to unit path'#000+
-  '**2FU<x>_set unit output path to <x>, ov','errides -FE'#000+
+  '**2FU<x>_set unit output path to <x','>, overrides -FE'#000+
   '*g1g<x>_generate debugger information:'#000+
   '*g1g<x>_generate debugger information:'#000+
   '*g2gg_use gsym'#000+
   '*g2gg_use gsym'#000+
   '*g2gd_use dbx'#000+
   '*g2gd_use dbx'#000+
@@ -571,7 +571,7 @@ const msgtxt : array[0..000103,1..240] of char=(
   '**1i_information'#000+
   '**1i_information'#000+
   '**2iD_return compiler date'#000+
   '**2iD_return compiler date'#000+
   '**2iV_return compiler version'#000+
   '**2iV_return compiler version'#000+
-  '**2iSO_return compiler OS',#000+
+  '**2iSO_return compil','er OS'#000+
   '**2iSP_return compiler processor'#000+
   '**2iSP_return compiler processor'#000+
   '**2iTO_return target OS'#000+
   '**2iTO_return target OS'#000+
   '**2iTP_return target processor'#000+
   '**2iTP_return target processor'#000+
@@ -579,73 +579,73 @@ const msgtxt : array[0..000103,1..240] of char=(
   '**1k<x>_Pass <x> to the linker'#000+
   '**1k<x>_Pass <x> to the linker'#000+
   '**1l_write logo'#000+
   '**1l_write logo'#000+
   '**1n_don'#039't read the default config file'#000+
   '**1n_don'#039't read the default config file'#000+
-  '**1o<x>_change the name of the ','executable produced to <x>'#000+
+  '**1o<x>_change the name of',' the executable produced to <x>'#000+
   '**1pg_generate profile code for gprof (defines FPC_PROFILE)'#000+
   '**1pg_generate profile code for gprof (defines FPC_PROFILE)'#000+
   '*L1P_use pipes instead of creating temporary assembler files'#000+
   '*L1P_use pipes instead of creating temporary assembler files'#000+
   '**1S<x>_syntax options:'#000+
   '**1S<x>_syntax options:'#000+
   '**2S2_switch some Delphi 2 extensions on'#000+
   '**2S2_switch some Delphi 2 extensions on'#000+
-  '**2Sc_supports operators li','ke C (*=,+=,/= and -=)'#000+
+  '**2Sc_supports operato','rs like C (*=,+=,/= and -=)'#000+
   '**2Sd_tries to be Delphi compatible'#000+
   '**2Sd_tries to be Delphi compatible'#000+
   '**2Se<x>_compiler stops after the <x> errors (default is 1)'#000+
   '**2Se<x>_compiler stops after the <x> errors (default is 1)'#000+
   '**2Sg_allow LABEL and GOTO'#000+
   '**2Sg_allow LABEL and GOTO'#000+
   '**2Sh_Use ansistrings'#000+
   '**2Sh_Use ansistrings'#000+
   '**2Si_support C++ styled INLINE'#000+
   '**2Si_support C++ styled INLINE'#000+
-  '**2Sm_support macros like C (global)'#000+
-  '**2','So_tries to be TP/BP 7.0 compatible'#000+
+  '**2Sm_support macros like C (global',')'#000+
+  '**2So_tries to be TP/BP 7.0 compatible'#000+
   '**2Sp_tries to be gpc compatible'#000+
   '**2Sp_tries to be gpc compatible'#000+
   '**2Ss_constructor name must be init (destructor must be done)'#000+
   '**2Ss_constructor name must be init (destructor must be done)'#000+
   '**2St_allow static keyword in objects'#000+
   '**2St_allow static keyword in objects'#000+
   '**1s_don'#039't call assembler and linker (only with -a)'#000+
   '**1s_don'#039't call assembler and linker (only with -a)'#000+
-  '**1u<x>_undefines t','he symbol <x>'#000+
+  '**1u<x>_undefi','nes the symbol <x>'#000+
   '**1U_unit options:'#000+
   '**1U_unit options:'#000+
   '**2Un_don'#039't check the unit name'#000+
   '**2Un_don'#039't check the unit name'#000+
   '**2Us_compile a system unit'#000+
   '**2Us_compile a system unit'#000+
   '**1v<x>_Be verbose. <x> is a combination of the following letters:'#000+
   '**1v<x>_Be verbose. <x> is a combination of the following letters:'#000+
   '**2*_e : Show errors (default)       d : Show debug info'#000+
   '**2*_e : Show errors (default)       d : Show debug info'#000+
-  '**2*_w : Show warnings ','              u : Show unit info'#000+
+  '**2*_w : Show warn','ings               u : Show unit info'#000+
   '**2*_n : Show notes                  t : Show tried/used files'#000+
   '**2*_n : Show notes                  t : Show tried/used files'#000+
   '**2*_h : Show hints                  m : Show defined macros'#000+
   '**2*_h : Show hints                  m : Show defined macros'#000+
   '**2*_i : Show general info           p : Show compiled procedures'#000+
   '**2*_i : Show general info           p : Show compiled procedures'#000+
-  '**2*_l : Show lin','enumbers            c : Show conditionals'#000+
+  '**2*_l : Sho','w linenumbers            c : Show conditionals'#000+
   '**2*_a : Show everything             0 : Show nothing (except errors)'#000+
   '**2*_a : Show everything             0 : Show nothing (except errors)'#000+
   '**2*_b : Show all procedure          r : Rhide/GCC compatibility mode'#000+
   '**2*_b : Show all procedure          r : Rhide/GCC compatibility mode'#000+
-  '**2*_    declarations if an error    x : Executable info (','Win32 only'+
+  '**2*_    declarations if an error    x : Executable i','nfo (Win32 only'+
   ')'#000+
   ')'#000+
   '**2*_    occurs'#000+
   '**2*_    occurs'#000+
   '**1X_executable options:'#000+
   '**1X_executable options:'#000+
   '*L2Xc_link with the c library'#000+
   '*L2Xc_link with the c library'#000+
   '**2Xs_strip all symbols from executable'#000+
   '**2Xs_strip all symbols from executable'#000+
   '**2XD_try to link dynamic          (defines FPC_LINK_DYNAMIC)'#000+
   '**2XD_try to link dynamic          (defines FPC_LINK_DYNAMIC)'#000+
-  '**2XS_try to link static (default) (defines FPC_LINK_ST','ATIC)'#000+
+  '**2XS_try to link static (default) (defines FPC_LI','NK_STATIC)'#000+
   '**2XX_try to link smart            (defines FPC_LINK_SMART)'#000+
   '**2XX_try to link smart            (defines FPC_LINK_SMART)'#000+
   '**0*_Processor specific options:'#000+
   '**0*_Processor specific options:'#000+
   '3*1A<x>_output format:'#000+
   '3*1A<x>_output format:'#000+
   '3*2Aas_assemble using GNU AS'#000+
   '3*2Aas_assemble using GNU AS'#000+
   '3*2Aasaout_assemble using GNU AS for aout (Go32v1)'#000+
   '3*2Aasaout_assemble using GNU AS for aout (Go32v1)'#000+
-  '3*2Anasmcoff_coff (Go32v2) file using ','Nasm'#000+
+  '3*2Anasmcoff_coff (Go32v2) file u','sing Nasm'#000+
   '3*2Anasmelf_elf32 (Linux) file using Nasm'#000+
   '3*2Anasmelf_elf32 (Linux) file using Nasm'#000+
   '3*2Anasmobj_obj file using Nasm'#000+
   '3*2Anasmobj_obj file using Nasm'#000+
   '3*2Amasm_obj file using Masm (Microsoft)'#000+
   '3*2Amasm_obj file using Masm (Microsoft)'#000+
   '3*2Atasm_obj file using Tasm (Borland)'#000+
   '3*2Atasm_obj file using Tasm (Borland)'#000+
   '3*2Acoff_coff (Go32v2) using internal writer'#000+
   '3*2Acoff_coff (Go32v2) using internal writer'#000+
-  '3*2Apecoff_pecoff (Win32) using inte','rnal writer'#000+
+  '3*2Apecoff_pecoff (Win32) using',' internal writer'#000+
   '3*1R<x>_assembler reading style:'#000+
   '3*1R<x>_assembler reading style:'#000+
   '3*2Ratt_read AT&T style assembler'#000+
   '3*2Ratt_read AT&T style assembler'#000+
   '3*2Rintel_read Intel style assembler'#000+
   '3*2Rintel_read Intel style assembler'#000+
   '3*2Rdirect_copy assembler text directly to assembler file'#000+
   '3*2Rdirect_copy assembler text directly to assembler file'#000+
   '3*1O<x>_optimizations:'#000+
   '3*1O<x>_optimizations:'#000+
   '3*2Og_generate smaller code'#000+
   '3*2Og_generate smaller code'#000+
-  '3*2OG_generate ','faster code (default)'#000+
+  '3*2OG_gene','rate faster code (default)'#000+
   '3*2Or_keep certain variables in registers (still BUGGY!!!)'#000+
   '3*2Or_keep certain variables in registers (still BUGGY!!!)'#000+
   '3*2Ou_enable uncertain optimizations (see docs)'#000+
   '3*2Ou_enable uncertain optimizations (see docs)'#000+
   '3*2O1_level 1 optimizations (quick optimizations)'#000+
   '3*2O1_level 1 optimizations (quick optimizations)'#000+
-  '3*2O2_level 2 optimizations (-O1 + slower optimizations)'#000+
-  '3*2O','3_level 3 optimizations (same as -O2u)'#000+
+  '3*2O2_level 2 optimizations (-O1 + slower optimizations)',#000+
+  '3*2O3_level 3 optimizations (same as -O2u)'#000+
   '3*2Op<x>_target processor:'#000+
   '3*2Op<x>_target processor:'#000+
   '3*3Op1_set target processor to 386/486'#000+
   '3*3Op1_set target processor to 386/486'#000+
   '3*3Op2_set target processor to Pentium/PentiumMMX (tm)'#000+
   '3*3Op2_set target processor to Pentium/PentiumMMX (tm)'#000+
   '3*3Op3_set target processor to PPro/PII/c6x86/K6 (tm)'#000+
   '3*3Op3_set target processor to PPro/PII/c6x86/K6 (tm)'#000+
-  '3*1T<x>_Target operating s','ystem:'#000+
+  '3*1T<x>_Target operat','ing system:'#000+
   '3*2TGO32V1_version 1 of DJ Delorie DOS extender'#000+
   '3*2TGO32V1_version 1 of DJ Delorie DOS extender'#000+
   '3*2TGO32V2_version 2 of DJ Delorie DOS extender'#000+
   '3*2TGO32V2_version 2 of DJ Delorie DOS extender'#000+
   '3*2TLINUX_Linux'#000+
   '3*2TLINUX_Linux'#000+
@@ -653,7 +653,7 @@ const msgtxt : array[0..000103,1..240] of char=(
   '3*2TWin32_Windows 32 Bit'#000+
   '3*2TWin32_Windows 32 Bit'#000+
   '6*1A<x>_output format'#000+
   '6*1A<x>_output format'#000+
   '6*2Aas_Unix o-file using GNU AS'#000+
   '6*2Aas_Unix o-file using GNU AS'#000+
-  '6*2Agas_GNU Motorola asse','mbler'#000+
+  '6*2Agas_GNU Motorola',' assembler'#000+
   '6*2Amit_MIT Syntax (old GAS)'#000+
   '6*2Amit_MIT Syntax (old GAS)'#000+
   '6*2Amot_Standard Motorola assembler'#000+
   '6*2Amot_Standard Motorola assembler'#000+
   '6*1O_optimizations:'#000+
   '6*1O_optimizations:'#000+
@@ -661,7 +661,7 @@ const msgtxt : array[0..000103,1..240] of char=(
   '6*2Og_generate smaller code'#000+
   '6*2Og_generate smaller code'#000+
   '6*2OG_generate faster code (default)'#000+
   '6*2OG_generate faster code (default)'#000+
   '6*2Ox_optimize maximum (still BUGGY!!!)'#000+
   '6*2Ox_optimize maximum (still BUGGY!!!)'#000+
-  '6*2O2_set target',' processor to a MC68020+'#000+
+  '6*2O2_set t','arget processor to a MC68020+'#000+
   '6*1R<x>_assembler reading style:'#000+
   '6*1R<x>_assembler reading style:'#000+
   '6*2RMOT_read motorola style assembler'#000+
   '6*2RMOT_read motorola style assembler'#000+
   '6*1T<x>_Target operating system:'#000+
   '6*1T<x>_Target operating system:'#000+
@@ -670,6 +670,6 @@ const msgtxt : array[0..000103,1..240] of char=(
   '6*2TMACOS_Macintosh m68k'#000+
   '6*2TMACOS_Macintosh m68k'#000+
   '6*2TLINUX_Linux-68k'#000+
   '6*2TLINUX_Linux-68k'#000+
   '**1*_'#000+
   '**1*_'#000+
-  '**1?_sho','ws this help'#000+
+  '**1','?_shows this help'#000+
   '**1h_shows this help without waiting'#000
   '**1h_shows this help without waiting'#000
 );
 );

+ 8 - 1
compiler/pexpr.pas

@@ -669,6 +669,7 @@ unit pexpr;
                          if getprocvar then
                          if getprocvar then
                            handle_procvar(pprocvardef(ppropertysym(sym)^.proptype),p2);
                            handle_procvar(pprocvardef(ppropertysym(sym)^.proptype),p2);
                          p1^.left:=gencallparanode(p2,p1^.left);
                          p1^.left:=gencallparanode(p2,p1^.left);
+                         p1^.isproperty:=true;
                          getprocvar:=false;
                          getprocvar:=false;
                        end;
                        end;
                      varsym :
                      varsym :
@@ -685,6 +686,7 @@ unit pexpr;
                               p1:=gensubscriptnode(pvarsym(plist^.sym),p1);
                               p1:=gensubscriptnode(pvarsym(plist^.sym),p1);
                             plist:=plist^.next;
                             plist:=plist^.next;
                           end;
                           end;
+                         p1^.isproperty:=true;
                          consume(_ASSIGNMENT);
                          consume(_ASSIGNMENT);
                          { read the expression }
                          { read the expression }
                          p2:=comp_expr(true);
                          p2:=comp_expr(true);
@@ -724,6 +726,7 @@ unit pexpr;
                                p1:=gensubscriptnode(pvarsym(plist^.sym),p1);
                                p1:=gensubscriptnode(pvarsym(plist^.sym),p1);
                              plist:=plist^.next;
                              plist:=plist^.next;
                            end;
                            end;
+                          p1^.isproperty:=true;
                        end;
                        end;
                      procsym :
                      procsym :
                        begin
                        begin
@@ -734,6 +737,7 @@ unit pexpr;
                           p1^.procdefinition:=pprocdef(ppropertysym(sym)^.readaccessdef);
                           p1^.procdefinition:=pprocdef(ppropertysym(sym)^.readaccessdef);
                           { insert paras }
                           { insert paras }
                           p1^.left:=paras;
                           p1^.left:=paras;
+                          p1^.isproperty:=true;
                        end
                        end
                      else
                      else
                        begin
                        begin
@@ -2113,7 +2117,10 @@ _LECKKLAMMER : begin
 end.
 end.
 {
 {
   $Log$
   $Log$
-  Revision 1.149  1999-10-22 10:39:34  peter
+  Revision 1.150  1999-10-22 14:37:30  peter
+    * error when properties are passed to var parameters
+
+  Revision 1.149  1999/10/22 10:39:34  peter
     * split type reading from pdecl to ptype unit
     * split type reading from pdecl to ptype unit
     * parameter_dec routine is now used for procedure and procvars
     * parameter_dec routine is now used for procedure and procvars
 
 

+ 4 - 2
compiler/ptype.pas

@@ -349,7 +349,6 @@ uses
            p,p2 : ppropertysym;
            p,p2 : ppropertysym;
            overriden : psym;
            overriden : psym;
            hs : string;
            hs : string;
-           code : integer;
            varspez : tvarspez;
            varspez : tvarspez;
            sc : pstringcontainer;
            sc : pstringcontainer;
            hp : pdef;
            hp : pdef;
@@ -1609,7 +1608,10 @@ uses
 end.
 end.
 {
 {
   $Log$
   $Log$
-  Revision 1.1  1999-10-22 10:39:35  peter
+  Revision 1.2  1999-10-22 14:37:30  peter
+    * error when properties are passed to var parameters
+
+  Revision 1.1  1999/10/22 10:39:35  peter
     * split type reading from pdecl to ptype unit
     * split type reading from pdecl to ptype unit
     * parameter_dec routine is now used for procedure and procvars
     * parameter_dec routine is now used for procedure and procvars
 
 

+ 8 - 1
compiler/tccal.pas

@@ -200,6 +200,10 @@ implementation
               { check if local proc/func is assigned to procvar }
               { check if local proc/func is assigned to procvar }
               if p^.left^.resulttype^.deftype=procvardef then
               if p^.left^.resulttype^.deftype=procvardef then
                 test_local_to_procvar(pprocvardef(p^.left^.resulttype),defcoll^.data);
                 test_local_to_procvar(pprocvardef(p^.left^.resulttype),defcoll^.data);
+              { property is not allowed as var parameter }
+              if (defcoll^.paratyp=vs_var) and
+                 (p^.left^.isproperty) then
+                CGMessagePos(p^.left^.fileinfo,type_e_argument_cant_be_assigned);
               { generate the high() value tree }
               { generate the high() value tree }
               if push_high_param(defcoll^.data) then
               if push_high_param(defcoll^.data) then
                 gen_high_tree(p,is_open_string(defcoll^.data));
                 gen_high_tree(p,is_open_string(defcoll^.data));
@@ -1224,7 +1228,10 @@ implementation
 end.
 end.
 {
 {
   $Log$
   $Log$
-  Revision 1.68  1999-10-13 10:35:27  peter
+  Revision 1.69  1999-10-22 14:37:30  peter
+    * error when properties are passed to var parameters
+
+  Revision 1.68  1999/10/13 10:35:27  peter
     * var must match exactly error msg extended with got and expected type
     * var must match exactly error msg extended with got and expected type
     * array constructor type check now gives error on wrong types
     * array constructor type check now gives error on wrong types
 
 

+ 19 - 10
compiler/tcinl.pas

@@ -582,8 +582,9 @@ implementation
                       if codegenerror then
                       if codegenerror then
                        exit;
                        exit;
                       { first param must be var }
                       { first param must be var }
-                      if p^.left^.left^.location.loc<>LOC_REFERENCE then
-                        CGMessage(type_e_argument_must_be_lvalue);
+                      if (p^.left^.left^.location.loc<>LOC_REFERENCE) or
+                         (p^.left^.left^.isproperty) then
+                        CGMessagePos(p^.left^.left^.fileinfo,type_e_argument_cant_be_assigned);
                       { check type }
                       { check type }
                       if (p^.left^.resulttype^.deftype in [enumdef,pointerdef]) or
                       if (p^.left^.resulttype^.deftype in [enumdef,pointerdef]) or
                          is_ordinal(p^.left^.resulttype) then
                          is_ordinal(p^.left^.resulttype) then
@@ -851,9 +852,12 @@ implementation
                   { valid string ? }
                   { valid string ? }
                   if not assigned(hp) or
                   if not assigned(hp) or
                      (hp^.left^.resulttype^.deftype<>stringdef) or
                      (hp^.left^.resulttype^.deftype<>stringdef) or
-                     (hp^.right=nil) or
-                     (hp^.left^.location.loc<>LOC_REFERENCE) then
+                     (hp^.right=nil) then
                     CGMessage(cg_e_illegal_expression);
                     CGMessage(cg_e_illegal_expression);
+                  { we need a var parameter }
+                  if (hp^.left^.location.loc<>LOC_REFERENCE) or
+                     (hp^.left^.isproperty) then
+                    CGMessagePos(p^.left^.fileinfo,type_e_argument_cant_be_assigned);
                   { generate the high() value for the shortstring }
                   { generate the high() value for the shortstring }
                   if is_shortstring(hp^.left^.resulttype) then
                   if is_shortstring(hp^.left^.resulttype) then
                     gen_high_tree(hp,true);
                     gen_high_tree(hp,true);
@@ -948,7 +952,8 @@ implementation
                        if codegenerror then exit;
                        if codegenerror then exit;
                        p^.left^.right := hp;
                        p^.left^.right := hp;
                      {code has to be a var parameter}
                      {code has to be a var parameter}
-                       if (p^.left^.left^.location.loc<>LOC_REFERENCE) then
+                       if (p^.left^.left^.location.loc<>LOC_REFERENCE) or
+                          (p^.left^.left^.isproperty) then
                          CGMessage(type_e_variable_id_expected)
                          CGMessage(type_e_variable_id_expected)
                        else
                        else
                          if (p^.left^.left^.resulttype^.deftype <> orddef) or
                          if (p^.left^.left^.resulttype^.deftype <> orddef) or
@@ -973,7 +978,8 @@ implementation
                   if (hpp^.left^.treetype=funcretn) then
                   if (hpp^.left^.treetype=funcretn) then
                    procinfo^.funcret_is_valid:=true;
                    procinfo^.funcret_is_valid:=true;
                   hpp^.right := hp;
                   hpp^.right := hp;
-                  if (hpp^.left^.location.loc<>LOC_REFERENCE) then
+                  if (hpp^.left^.location.loc<>LOC_REFERENCE) or
+                     (hpp^.left^.isproperty) then
                     CGMessage(type_e_variable_id_expected)
                     CGMessage(type_e_variable_id_expected)
                   else
                   else
                     If Not((hpp^.left^.resulttype^.deftype = floatdef) or
                     If Not((hpp^.left^.resulttype^.deftype = floatdef) or
@@ -1023,9 +1029,9 @@ implementation
                       if (p^.left^.left^.treetype=funcretn) then
                       if (p^.left^.left^.treetype=funcretn) then
                        procinfo^.funcret_is_valid:=true;
                        procinfo^.funcret_is_valid:=true;
                       { first param must be var }
                       { first param must be var }
-                      if (p^.left^.left^.location.loc<>LOC_REFERENCE) and
-                         (p^.left^.left^.location.loc<>LOC_CREGISTER) then
-                        CGMessage(cg_e_illegal_expression);
+                      if not(p^.left^.left^.location.loc in [LOC_REFERENCE,LOC_CREGISTER]) or
+                         (p^.left^.left^.isproperty) then
+                        CGMessagePos(p^.left^.left^.fileinfo,type_e_argument_cant_be_assigned);
                       { check type }
                       { check type }
                       if (p^.left^.resulttype^.deftype=setdef) then
                       if (p^.left^.resulttype^.deftype=setdef) then
                         begin
                         begin
@@ -1262,7 +1268,10 @@ implementation
 end.
 end.
 {
 {
   $Log$
   $Log$
-  Revision 1.54  1999-10-21 16:41:41  florian
+  Revision 1.55  1999-10-22 14:37:31  peter
+    * error when properties are passed to var parameters
+
+  Revision 1.54  1999/10/21 16:41:41  florian
     * problems with readln fixed: esi wasn't restored correctly when
     * problems with readln fixed: esi wasn't restored correctly when
       reading ordinal fields of objects futher the register allocation
       reading ordinal fields of objects futher the register allocation
       didn't take care of the extra register when reading ordinal values
       didn't take care of the extra register when reading ordinal values

+ 5 - 1
compiler/tree.pas

@@ -196,6 +196,7 @@ unit tree;
           resulttype : pdef;
           resulttype : pdef;
           fileinfo : tfileposinfo;
           fileinfo : tfileposinfo;
           localswitches : tlocalswitches;
           localswitches : tlocalswitches;
+          isproperty : boolean;
 {$ifdef extdebug}
 {$ifdef extdebug}
           firstpasscount : longint;
           firstpasscount : longint;
 {$endif extdebug}
 {$endif extdebug}
@@ -1790,7 +1791,10 @@ unit tree;
 end.
 end.
 {
 {
   $Log$
   $Log$
-  Revision 1.99  1999-09-27 23:45:03  peter
+  Revision 1.100  1999-10-22 14:37:31  peter
+    * error when properties are passed to var parameters
+
+  Revision 1.99  1999/09/27 23:45:03  peter
     * procinfo is now a pointer
     * procinfo is now a pointer
     * support for result setting in sub procedure
     * support for result setting in sub procedure