2001-09-27 Miguel de Icaza <miguel@ximian.com>
[mono.git] / mcs / mcs / ChangeLog
1 2001-09-27  Miguel de Icaza  <miguel@ximian.com>
2
3         * typemanager.cs (InitCoreTypes): Init also value_type
4         (System.ValueType). 
5
6         * expression.cs (Cast::Resolve): First resolve the child expression.
7
8         (LValue): Add new method AddressOf to be used by
9         the `&' operator.  
10
11         Change the argument of Store to take an EmitContext instead of an
12         ILGenerator, because things like FieldExpr need to be able to call
13         their children expression to generate the instance code. 
14
15         (Expression::Error, Expression::Warning): Sugar functions for
16         reporting errors.
17
18         (Expression::MemberLookup): Accept a TypeContainer instead of a
19         Report as the first argument.
20
21         (Expression::ResolvePrimary): Killed.  I still want to improve
22         this as currently the code is just not right.
23
24         (Expression::ResolveMemberAccess): Simplify, but it is still
25         wrong. 
26
27         (Unary::Resolve): Catch errors in AddressOf operators.
28
29         (LocalVariableReference::Emit, ::Store, ::AddressOf): typecast
30         index to a byte for the short-version, or the compiler will choose
31         the wrong Emit call, which generates the wrong data.
32
33         (ParameterReference::Emit, ::Store): same.
34
35         (FieldExpr::AddressOf): Implement.
36         
37         * typemanager.cs: TypeManager: made public variable instead of
38         property.
39         
40         * driver.cs: document --fatal.
41
42         * report.cs (ErrorMessage, WarningMessage): new names for the old
43         Error and Warning classes.
44
45         * cs-parser.jay (member_access): Turn built-in access to types
46         into a normal simplename
47
48 2001-09-27  Ravi Pratap  <ravi@ximian.com>
49
50         * expression.cs (Invocation::BetterConversion): Fix to cope
51         with q being null, since this was introducing a bug.
52
53         * expression.cs (ConvertImplicit): Do built-in conversions first.
54
55 2001-09-27  Ravi Pratap  <ravi@ximian.com>
56
57         * expression.cs (UserImplicitCast::Resolve): Fix bug.
58
59 2001-09-27  Ravi Pratap  <ravi@ximian.com>
60
61         * class.cs (TypeContainer::AddConstructor): Fix a stupid bug
62         I had introduced long ago (what's new ?).
63
64         * expression.cs (UserImplicitCast::CanConvert): Static method to do 
65         the work of all the checking. 
66         (ConvertImplicit): Call CanConvert and only then create object if necessary.
67         (UserImplicitCast::CanConvert, ::Resolve): Re-write.
68
69         (Unary::Operator): Rename Add and Subtract to Addition and Subtraction because
70         that is the right way. 
71
72         (Invocation::MakeUnionSet): Convenience function to make unions of sets for 
73         overloading resolution. Use everywhere instead of cutting and pasting code.
74
75         (Binary::ResolveOperator): Use MakeUnionSet.
76
77         (UserImplicitCast::CanConvert, ::Resolve): Update to take care of the case when 
78         we have to convert to bool types. Not complete yet.
79         
80 2001-09-27  Miguel de Icaza  <miguel@ximian.com>
81
82         * typemanager.cs (TypeManager::CSharpName): support ushort.
83
84         * expression.cs (Expression::TryImplicitIntConversion): Attempts
85         to provide an expression that performsn an implicit constant int
86         conversion (section 6.1.6).
87         (Expression::ConvertImplicitRequired): Reworked to include
88         implicit constant expression conversions.
89
90         (Expression::ConvertNumericExplicit): Finished.
91
92         (Invocation::Emit): If InstanceExpression is null, then it means
93         that we perform a call on this.
94         
95 2001-09-26  Miguel de Icaza  <miguel@ximian.com>
96
97         * expression.cs (Unary::Emit): Remove some dead code.
98         (Probe): Implement Resolve and Emit for `is'.
99         (Expression::ConvertImplicitRequired): Attempt to do constant
100         expression conversions here.  Maybe should be moved to
101         ConvertImplicit, but I am not sure.
102         (Expression::ImplicitLongConstantConversionPossible,
103         Expression::ImplicitIntConstantConversionPossible): New functions
104         that tell whether is it possible to apply an implicit constant
105         expression conversion.
106
107         (ConvertNumericExplicit): Started work on explicit numeric
108         conversions.
109
110         * cs-parser.jay: Update operator constants.
111
112         * parameter.cs (Parameters::GetParameterInfo): Hook up VerifyArgs
113         (Parameters::GetSignature): Hook up VerifyArgs here.
114         (Parameters::VerifyArgs): Verifies that no two arguments have the
115         same name. 
116
117         * class.cs (Operator): Update the operator names to reflect the
118         ones that the spec expects (as we are just stringizing the
119         operator names).
120         
121         * expression.cs (Unary::ResolveOperator): Fix bug: Use
122         MethodInfo's ReturnType instead of LookupMethodByBuilder as the
123         previous usage did only work for our methods.
124         (Expression::ConvertImplicit): Handle decimal implicit numeric
125         conversions as well.
126         (Expression::InternalTypeConstructor): Used to invoke constructors
127         on internal types for default promotions.
128
129         (Unary::Emit): Implement special handling for the pre/post
130         increment/decrement for overloaded operators, as they need to have
131         the same semantics as the other operators.
132
133         (Binary::ResolveOperator): ditto.
134         (Invocation::ConversionExists): ditto.
135         (UserImplicitCast::Resolve): ditto.
136         
137 2001-09-26  Ravi Pratap  <ravi@ximian.com>
138
139         * expression.cs (Unary::Emit and Binary::Emit): If we have an overloaded
140         operator, return after emitting body. Regression tests pass again !
141
142         * expression.cs (ConvertImplicit): Take TypeContainer as first argument
143         (Unary::ForceConversion, Binary::ForceConversion): Ditto.
144         (Invocation::OverloadResolve): Ditto.
145         (Invocation::BetterFunction, BetterConversion, ConversionExists): Ditto.
146
147         * everywhere : update calls to the above methods accordingly.
148
149 2001-09-26  Miguel de Icaza  <miguel@ximian.com>
150
151         * assign.cs (Assign): Make it inherit from ExpressionStatement.
152
153         * expression.cs (ExpressionStatement): New base class used for
154         expressions that can appear in statements, so that we can provide
155         an alternate path to generate expression that do not leave a value
156         on the stack.
157
158         (Expression::Emit, and all the derivatives): We no longer return
159         whether a value is left on the stack or not.  Every expression
160         after being emitted leaves a single value on the stack.
161
162         * codegen.cs (EmitContext::EmitStatementExpression): Use the
163         facilties of ExpressionStatement if possible.
164
165         * cs-parser.jay: Update statement_expression.
166
167 2001-09-25  Miguel de Icaza  <miguel@ximian.com>
168
169         * driver.cs: Change the wording of message
170
171 2001-09-25  Ravi Pratap  <ravi@ximian.com>
172
173         * expression.cs (Binary::ResolveOperator): Had forgottten to set 
174         the type of the expression to the return type of the method if
175         we have an overloaded operator match ! The regression tests pass again !
176         (Unary::ResolveOperator): Ditto.
177
178         * expression.cs (Invocation::ConversionExists): Correct the member lookup
179         to find "op_Implicit", not "implicit" ;-)
180         (UserImplicitCast): New class to take care of user-defined implicit conversions.
181         (ConvertImplicit, ForceConversion): Take TypeContainer argument
182
183         * everywhere : Correct calls to the above accordingly.
184
185         * expression.cs (UserImplicitCast::Resolve, ::Emit): Implement.
186         (ConvertImplicit): Do user-defined conversion if it exists.
187
188 2001-09-24  Miguel de Icaza  <miguel@ximian.com>
189
190         * assign.cs: track location.
191         (Resolve): Use implicit conversions on assignment.
192
193         * literal.cs: Oops.  Not good, Emit of short access values should
194         pass (Bytes) or the wrong argument will be selected.
195
196         * expression.cs (Unary::Emit): Emit code for -expr.
197         
198         (Unary::ResolveOperator): Handle `Substract' for non-constants
199         (substract from zero from the non-constants).
200         Deal with Doubles as well. 
201         
202         (Expression::ConvertImplicitRequired): New routine that reports an
203         error if no implicit conversion exists. 
204
205         (Invocation::OverloadResolve): Store the converted implicit
206         expressions if we make them
207         
208 2001-09-24  Ravi Pratap  <ravi@ximian.com>
209
210         * class.cs (ConstructorInitializer): Take a Location argument.
211         (ConstructorBaseInitializer): Same here.
212         (ConstructorThisInitializer): Same here.
213
214         * cs-parser.jay : Update all calls accordingly.
215
216         * expression.cs (Unary, Binary, New): Take location argument.
217         Update accordingly everywhere.
218
219         * cs-parser.jay : Update all calls to the above to take a location
220         argument.
221
222         * class.cs : Ditto.
223
224 2001-09-24  Ravi Pratap  <ravi@ximian.com>
225
226         * expression.cs (Invocation::BetterFunction): Take TypeContainer argument
227         (Invocation::BetterConversion): Same here
228         (Invocation::ConversionExists): Ditto.
229
230         (Invocation::ConversionExists): Implement.
231
232 2001-09-22  Ravi Pratap  <ravi@ximian.com>
233
234         * expression.cs (OverloadResolve): Improve some more to catch errors 1502 and 1503
235         Also take an additional TypeContainer argument.
236
237         * All over : Pass in TypeContainer as argument to OverloadResolve.
238
239         * typemanager.cs (CSharpName): Update to check for the string type and return
240         that too.
241
242         * expression.cs (Invocation::FullMethodDesc): New static method to return a string fully describing
243         a given method.
244         
245 2001-09-21  Ravi Pratap  <ravi@ximian.com>
246
247         * expression.cs (Invocation::OverloadResolve): Re-write to conform more to the spec.
248         (Invocation::BetterFunction): Implement.
249         (Invocation::BetterConversion): Implement.
250         (Invocation::ConversionExists): Skeleton, no implementation yet.
251
252         Okay, things work fine !
253
254 2001-09-21  Miguel de Icaza  <miguel@ximian.com>
255
256         * typemanager.cs: declare and load enum_type, delegate_type and
257         void_type. 
258
259         * expression.cs (Expression::Emit): Now emit returns a value that
260         tells whether a value is left on the stack or not.  This strategy
261         might be reveted tomorrow with a mechanism that would address
262         multiple assignments.
263         (Expression::report118): Utility routine to report mismatches on
264         the ExprClass.
265
266         (Unary::Report23): Report impossible type/operator combination
267         utility function.
268
269         (Unary::IsIncrementableNumber): Whether the type can be
270         incremented or decremented with add.
271         (Unary::ResolveOperator): Also allow enumerations to be bitwise
272         complemented. 
273         (Unary::ResolveOperator): Implement ++, !, ~, ++ and --.
274
275         (Invocation::Emit): Deal with new Emit convetion.
276         
277         * All Expression derivatives: Updated their Emit method to return
278         whether they leave values on the stack or not.
279         
280         * codegen.cs (CodeGen::EmitStatement): Pop values left on the
281         stack for expressions that are statements. 
282
283 2001-09-20  Miguel de Icaza  <miguel@ximian.com>
284
285         * expression.cs (LValue): New interface.  Must be implemented by
286         LValue objects.
287         (LocalVariableReference, ParameterReference, FieldExpr): Implement
288         LValue interface.
289         
290         * assign.cs (Assign::Emit, Assign::Resolve): Use new LValue
291         interface for generating code, simplifies the code.
292
293 2001-09-20  Ravi Pratap  <ravi@ximian.com>
294
295         * expression.cs (everywhere): Comment out return statements in ::Resolve
296         methods to avoid the warnings.
297
298 2001-09-20  Miguel de Icaza  <miguel@ximian.com>
299
300         * driver.cs (parse): Report error 2001 if we can not open the
301         source file.
302
303         * expression.cs (SimpleName::ResolveSimpleName): Error if we can
304         not resolve it.
305
306         * cs-parser.jay (QualifierIdentifier): Pass location to SimpleName
307         object. 
308
309         * statement.cs (Block::EmitMeta): Reuse the count across all the variables,
310         otherwise nested blocks end up with the same index.
311
312         * codegen.cs (CodeGen::EmitTopBlock): Pass initial sequence
313
314         * expression.cs:  Instead of having FIXMEs in the Resolve
315         functions, throw exceptions so it is obvious that we are facing a
316         bug. 
317
318         * cs-parser.jay (invocation_expression): Pass Location information.
319
320         * codegen.cs (CodeGen::Save, CodeGen::CodeGen, CodeGen::Basename):
321         Use a basename for those routines because .NET does not like paths
322         on them. 
323
324         * class.cs (TypeContainer::AddMethod): Do not call DefineName if the name was
325         already defined.
326
327 2001-09-19  Miguel de Icaza  <miguel@ximian.com>
328
329         * typemanager.cs (TypeManager::CoreLookupType): A function to make sure that we
330         are loading the correct data types (throws an exception if not).
331         (TypeManager::InitCoreTypes): Use CoreLookupType
332
333         * expression.cs (Unary::ResolveOperator): return the child
334         expression for expressions which are just +expr.
335         (Unary::ResolveOperator): Return negative literals for -LITERAL
336         expressions (otherwise they are Unary {Literal}).
337         (Invocation::Badness): Take into account `Implicit constant
338         expression conversions'.
339
340         * literal.cs (LongLiteral): Implement long literal class.
341         (IntLiteral): export the `Value' of the intliteral. 
342
343 2001-09-19  Ravi Pratap  <ravi@ximian.com>
344
345         * expression.cs (Binary::Emit): Finally get the emission right ! Woo!
346
347         * class.cs (Operator::Define): Change the methodname prefix to 'op_' 
348         instead of 'Operator'
349
350         * expression.cs (Binary::ResolveOperator): Update accordingly.
351         (Unary::Operator): Change names to 'Add' and 'Subtract' instead 'Plus'
352         and 'Minus'
353
354         * cs-parser.jay (unary_expression): Update to use the new names.
355
356         * gen-treedump.cs (GetUnary): Same here.
357
358         * expression.cs (Unary::Resolve): Implement.
359         (Binary::ResolveOperator): Re-write bits to quietly continue if no overloaded 
360         operators are found instead of making noise ;-)
361         (Unary::ResolveOperator): New method to do precisely the same thing which
362         Binary::ResolveOperator does for Binary expressions.
363         (Unary.method, .Arguments): Add.
364         (Unary::OperName): Implement.   
365         (Unary::ForceConversion): Copy and Paste !
366
367         * class.cs (Operator::Define): Fix a small bug for the case when we have 
368         a unary operator.
369
370         * expression.cs (Unary::Emit): Implement. Need to find the right Opcodes
371         for the inbuilt operators. Only overloading works for now ;-)
372
373 2001-09-18  Miguel de Icaza  <miguel@ximian.com>
374
375         * expression.cs (CheckedExpr::Resolve, CheckedExpr::Emit,
376         UnCheckedExpr::Resolve, UnCheckedExpr::Emit): Implement.
377
378         * expression.cs (This::Emit): Implement. 
379         (This::Resolve): Implement.
380         (TypeOf:Resolve): Implement.
381         (Expression::ResolveSimpleName): Add an implicit this to instance
382         field references. 
383         (MemberAccess::Resolve): Deal with Parameters and Fields. 
384         Bind instance variable to Field expressions.
385         (FieldExpr::Instance): New field used to track the expression that
386         represents the object instance.
387         (FieldExpr::Resolve): Track potential errors from MemberLookup not
388         binding 
389         (FieldExpr::Emit): Implement.
390
391         * codegen.cs (EmitIf, EmitStatement, EmitBlock): Propagate whether
392         the last instruction contains a return opcode to avoid generating
393         the last `ret' instruction (this generates correct code, and it is
394         nice to pass the peverify output).
395
396         * class.cs (TypeContainer::EmitFieldInitializers): Implement field
397         initializer for static and instance variables.
398         (Constructor::Emit): Allow initializer to be null in the case of
399         static constructors.  Only emit initializer for instance
400         constructors. 
401
402         (TypeContainer::FindMembers): Return a null array if there are no
403         matches.
404
405         Also fix the code for the MemberTypes.Method branch, as it was not
406         scanning that for operators (or tried to access null variables before).
407
408         * assign.cs (Assign::Emit): Handle instance and static fields. 
409
410         * TODO: Updated.
411
412         * driver.cs: Stop compilation if there are parse errors.
413
414         * cs-parser.jay (constructor_declaration): Provide default base
415         initializer for non-static constructors.
416         (constructor_declarator): Do not provide a default base
417         initializers if none was specified.
418         Catch the fact that constructors should not have parameters.
419
420         * class.cs: Do not emit parent class initializers for static
421         constructors, that should be flagged as an error.
422
423 2001-09-18  Ravi Pratap  <ravi@ximian.com>
424
425         * class.cs (RegisterMethodBuilder): Remove : it's unnecessary.
426         Move back code into TypeContainer::Populate.
427
428 2001-09-18  Ravi Pratap  <ravi@ximian.com>
429
430         * class.cs (TypeContainer::AddConstructor): Fix the check to
431         compare against Name, not Basename. 
432         (Operator::OpType): Change Plus and Minus to Add and Subtract.
433
434         * cs-parser.jay : Update accordingly.
435
436         * class.cs (TypeContainer::FindMembers): For the case where we are searching
437         for methods, don't forget to look into the operators too.
438         (RegisterMethodBuilder): Helper method to take care of this for
439         methods, constructors and operators.
440         (Operator::Define): Completely revamp.
441         (Operator.OperatorMethod, MethodName): New fields.
442         (TypeContainer::Populate): Move the registering of builders into
443         RegisterMethodBuilder.
444         (Operator::Emit): Re-write.
445
446         * expression.cs (Binary::Emit): Comment out code path to emit method
447         invocation stuff for the case when we have a user defined operator. I am
448         just not able to get it right !
449         
450 2001-09-17  Miguel de Icaza  <miguel@ximian.com>
451
452         * expression.cs (Expression::OverloadResolve): Drop TypeContainer
453         argument. 
454
455         (Expression::MemberLookup): Provide a version that allows to
456         specify the MemberTypes and BindingFlags. 
457
458         * statement.cs (Block::GetVariableInfo): Forgot to recurse here,
459         so it was not fetching variable information from outer blocks.
460
461         * modifiers.cs: (Modifiers::TypeAttr): Invert condition on
462         Beforefieldinit as it was buggy.
463
464         * rootcontext.cs (::LookupInterfaceOrClass): Removed an Error -200
465         that Ravi put here.  
466
467         * class.cs (Constructor::Emit): Only emit if block is not null.
468         (TypeContainer::EmitDefaultConstructor): Removed routine, now we
469         deal with this by semantically definining it as if the user had
470         done it.
471
472         (TypeContainer::FindMembers): Removed ad-hoc hack to deal with
473         constructors as we now "emit" them at a higher level.
474
475         (TypeContainer::DefineDefaultConstructor): Used to define the
476         default constructors if none was provided.
477
478         (ConstructorInitializer): Add methods Resolve and Emit. 
479         
480         * expression.cs: Cast to ConstructorInfo instead of MethodInfo
481
482 2001-09-17  Ravi Pratap  <ravi@ximian.com>
483
484         * class.cs (TypeContainer::EmitDefaultConstructor): Register
485         the default constructor builder with our hashtable for methodbuilders
486         to methodcores.
487
488         * expression.cs (Invocation::OverloadResolve): Add a check for pd == null
489         and argument_count is 0 in which case we have a match.
490         (Binary::ResolveOperator): More null checking and miscellaneous coding
491         style cleanup.
492
493 2001-09-17  Ravi Pratap  <ravi@ximian.com>
494
495         * rootcontext.cs (IsNameSpace): Compare against null.
496
497         * everywhere : Correct spelling to 'Greater' and to 'Subtract'
498
499         * class.cs (Operator::OpType): Change names to match the ones in Binary::Operator
500         and Unary::Operator.
501
502         * cs-parser.jay (operator_declaration, CheckBinaryOperator, CheckUnaryOperator): Update
503         accordingly.
504
505         * expression.cs (Binary::method): New member to hold the MethodBase for the case when
506         we have overloaded operators.
507         (Binary::ResolveOperator): Implement the part which does the operator overload
508         resolution.
509
510         * class.cs (Operator::Emit): Implement.
511         (TypeContainer::Emit): Emit the operators we have too.
512
513         * expression.cs (Binary::Emit): Update to emit the appropriate code for
514         the case when we have a user-defined operator.
515         
516 2001-09-17  Miguel de Icaza  <miguel@ximian.com>
517
518         * rootcontext.cs: Fix bug: tree.Namespaces might be null.
519
520 2001-09-16  Ravi Pratap  <ravi@ximian.com>
521
522         * class.cs (EmitStaticFieldInitializers, EmitFieldInitializers): Make public.
523         (TypeContainer::EmitConstructor): Remove and move code into Contructor::Emit.
524         (Constructor::Emit): Implement.
525         (EmitStaticFieldInitializers, EmitFieldInitializers): Ensure we return immediately
526         if we have no work to do. 
527         (TypeContainer::Emit): Pass in TypeContainer as argument to the constructor's 
528         Emit method.
529
530         * interface.cs (Interface::InterfaceAttr): Re-write to be more correct and complete.
531         (Interface::IsTopLevel): Add. Same as TypeContainer::IsTopLevel.
532
533         * class.cs (TypeContainer::IsTopLevel): Modify to use parent.Parent instead
534         of parent.parent.
535
536 2001-09-15  Ravi Pratap  <ravi@ximian.com>
537
538         * tree.cs (Tree::namespaces): New hashtable to keep track of namespaces
539         in the source.
540         (Tree::RecordNamespace): Method to do what the name says ;-)
541         (Tree::Namespaces): Property to get at the namespaces hashtable.
542
543         * cs-parser.jay (namespace_declaration): Call RecordNamespace to 
544         keep track.
545
546         * rootcontext.cs (IsNamespace): Fixed it :-)
547
548 2001-09-14  Miguel de Icaza  <miguel@ximian.com>
549
550         * class.cs (TypeContainer::FindMembers): Add support for
551         constructors. 
552         (MethodCore): New class that encapsulates both the shared aspects
553         of a Constructor and a Method.  
554         (Method, Constructor): Factored pieces into MethodCore.
555
556         * driver.cs: Added --fatal which makes errors throw exceptions.
557         Load System assembly as well as part of the standard library.
558
559         * report.cs: Allow throwing exceptions on errors for debugging.
560
561         * modifiers.cs: Do not use `parent', instead use the real type
562         container to evaluate permission settings.
563
564         * class.cs: Put Ravi's patch back in.  He is right, and we will
565         have to cope with the
566
567 2001-09-14  Ravi Pratap  <ravi@ximian.com>
568
569         * modifiers.cs (TypeAttr, MethodAttr, FieldAttr): Map protected internal to
570         FamORAssem, not FamANDAssem.
571         
572 2001-09-14  Miguel de Icaza  <miguel@ximian.com>
573
574         * driver.cs: Added --parse option that only parses its input files
575         and terminates.
576
577         * class.cs: Reverted last change from Ravi to IsTopLevel.  That is
578         incorrect.  IsTopLevel is not used to tell whether an object is
579         root_types or not (that can be achieved by testing this ==
580         root_types).  But to see if this is a top-level *class* (not
581         necessarly our "toplevel" container). 
582
583 2001-09-14  Ravi Pratap  <ravi@ximian.com>
584
585         * enum.cs (Enum::Define): Modify to call the Lookup method on the
586         parent instead of a direct call to GetType.
587
588 2001-09-14  Ravi Pratap  <ravi@ximian.com>
589
590         * class.cs (TypeContainer::TypeAttr): Remove property code and move it into
591         Modifiers.TypeAttr. This should just be a call to that method.
592
593         * modifiers.cs (TypeAttr): Re-write and take an extra argument, the TypeContainer
594         object so that we can determine if we are top-level or not.
595
596         * delegate.cs (Delegate::Define): Update call to TypeAttr method to pass in the 
597         TypeContainer too.
598
599         * enum.cs (Enum::Define): Ditto.
600
601         * modifiers.cs (FieldAttr): Re-write.
602
603         * class.cs (TypeContainer::IsTopLevel): Change accessibility to public.
604         (TypeContainer::HaveStaticConstructor): New property to provide access
605         to precisely that info.
606
607         * modifiers.cs (MethodAttr): Re-write.
608         (EventAttr): Remove altogether as there seems to be no ostensible use for it.
609
610         * class.cs (TypeContainer::IsTopLevel): Re-write. root_types doesn't seem to be the parent
611         of top-level types as claimed.
612         
613 2001-09-13  Miguel de Icaza  <miguel@ximian.com>
614
615         * expression.cs (MemberLookup): Fruitless attempt to lookup
616         constructors.  Maybe I need to emit default constructors?  That
617         might be it (currently .NET emits this for me automatically).
618         (Invocation::OverloadResolve): Cope with Arguments == null.
619         (Invocation::EmitArguments): new function, shared by the new
620         constructor and us.
621         (Invocation::Emit): Handle static and instance methods.  Emit
622         proper call instruction for virtual or non-virtual invocations.
623         (New::Emit): Implement.
624         (New::Resolve): Implement.
625         (MemberAccess:Resolve): Implement.
626         (MethodGroupExpr::InstanceExpression): used conforming to the spec
627         to track instances.
628         (FieldExpr::Resolve): Set type.
629
630         * support.cs: Handle empty arguments.
631                 
632         * cs-parser.jay (CompositeLookup, QualifierIdentifier,
633         SimpleLookup): Auxiliary routines to help parse a qualifier
634         identifier.  
635
636         Update qualifier_identifier rule.
637
638         * codegen.cs: Removed debugging messages.
639
640         * class.cs: Make this a global thing, this acts just as a "key" to
641         objects that we might have around.
642
643         (Populate): Only initialize method_builders_to_methods once.
644
645         * expression.cs (PropertyExpr): Initialize type from the
646         PropertyType. 
647
648         * codegen.cs (EmitContext::EmitBoolExpression): Use propper
649         Resolve pattern.  Attempt to implicitly convert value to boolean.
650         Emit code.
651
652         * expression.cs: Set the type for the int32/int32 argument case.
653         (Binary::ResolveOperator): Set the return type to boolean for
654         comparission operators
655
656         * typemanager.cs: Remove debugging print code.
657
658         (Invocation::Resolve): resolve type.
659
660         * class.cs: Allocate a MemberInfo of the correct size, as the code
661         elsewhere depends on the test to reflect the correct contents.
662
663         (Method::) Keep track of parameters, due to System.Reflection holes
664
665         (TypeContainer::Populate): Keep track of MethodBuilders to Method
666         mapping here.
667
668         (TypeContainer::FindMembers): Use ArrayList and then copy an array
669         of the exact size and return that.
670
671         (Class::LookupMethodByBuilder): New function that maps
672         MethodBuilders to its methods.  Required to locate the information
673         on methods because System.Reflection bit us again.
674
675         * support.cs: New file, contains an interface ParameterData and
676         two implementations: ReflectionParameters and InternalParameters
677         used to access Parameter information.  We will need to grow this
678         as required.
679
680         * expression.cs (Invocation::GetParameterData): implement a cache
681         and a wrapper around the ParameterData creation for methods. 
682         (Invocation::OverloadResolve): Use new code.
683
684 2001-09-13  Ravi Pratap  <ravi@ximian.com>
685
686         * class.cs (TypeContainer::EmitField): Remove and move into 
687         (Field::Define): here and modify accordingly.
688         (Field.FieldBuilder): New member.
689         (TypeContainer::Populate): Update accordingly.
690         (TypeContainer::FindMembers): Implement.
691
692 2001-09-13  Miguel de Icaza  <miguel@ximian.com>
693
694         * statement.cs: (VariableInfo::VariableType): New field to be
695         initialized with the full type once it is resolved. 
696
697 2001-09-12  Miguel de Icaza  <miguel@ximian.com>
698
699         * parameter.cs (GetParameterInfo): Use a type cache to compute
700         things only once, and to reuse this information
701
702         * expression.cs (LocalVariableReference::Emit): Implement.
703         (OpcodeCast::Emit): fix.
704
705         (ParameterReference::Resolve): Implement.
706         (ParameterReference::Emit): Implement.
707
708         * cs-parser.jay: Fix bug introduced by Ravi, variable initializers
709         that are expressions need to stay as Expressions.
710
711         * typemanager.cs (CSharpName): Returns the C# name of a type if
712         possible. 
713
714         * expression.cs (Expression::ConvertImplicit): New function that
715         implements implicit type conversions.
716
717         (Expression::ImplicitReferenceConversion): Implements implicit
718         reference conversions.
719
720         (EmptyCast): New type for transparent casts.
721
722         (OpcodeCast): New type for casts of types that are performed with
723         a sequence of bytecodes.
724         
725         (BoxedCast): New type used for casting value types into reference
726         types.  Emits a box opcode.
727
728         (Binary::DoNumericPromotions): Implements numeric promotions of
729         and computation of the Binary::Type.
730
731         (Binary::EmitBranchable): Optimization.
732
733         (Binary::Emit): Implement code emission for expressions.
734         
735         * typemanager.cs (TypeManager): Added two new core types: sbyte
736         and byte.
737
738 2001-09-12  Ravi Pratap  <ravi@ximian.com>
739
740         * class.cs (TypeContainer::FindMembers): Method which does exactly
741         what Type.FindMembers does, only we don't have to use reflection. No
742         implementation yet.
743
744         * typemanager.cs (typecontainers): New hashtable to hold the corresponding
745         typecontainer objects as we need to get at them.
746         (TypeManager::AddUserType): Overload to take an extra argument, the TypeContainer.
747
748         * rootcontext.cs : Correspondingly modify called to AddUserType to pass the
749         typecontainer object.
750
751         * expression.cs (MemberLookup): Modify signature to take a RootContext object instead
752         of just a Report object.
753
754 2001-09-11  Ravi Pratap  <ravi@ximian.com>
755
756         * class.cs (Event::Define): Go back to using the prefixes "add_" and
757         "remove_"
758         (TypeContainer::Populate): Now define the delegates of the type too.
759         (TypeContainer.Delegates): Property to access the list of delegates defined
760         in the type.
761
762         * delegates.cs (Delegate::Define): Implement partially.
763
764         * modifiers.cs (TypeAttr): Handle more flags.
765
766 2001-09-11  Ravi Pratap  <ravi@ximian.com>
767
768         * class.cs (Indexer::Define): Fix for loop iteration condition to be just <
769         and not <=
770         (Operator::Define): Re-write logic to get types by using the LookupType method
771         instead of blindly doing a Type.GetType ! How stupid can I get ;-) ?
772         (Indexer::Define): Ditto.
773         (Event::Define): Ditto.
774         (Property::Define): Ditto.
775         
776 2001-09-10  Ravi Pratap  <ravi@ximian.com>
777
778         * class.cs (TypeContainer::Populate): Now define operators too. 
779         (TypeContainer.Operators): New property to access the list of operators
780         in a type.
781         (Operator.OperatorMethodBuilder): New member to hold the method builder
782         for the operator we are defining.
783         (Operator::Define): Implement.
784
785 2001-09-10  Ravi Pratap  <ravi@ximian.com>
786
787         * class.cs (Event::Define): Make the prefixes of the accessor methods
788         addOn_ and removeOn_ 
789
790         * genericparser.cs (GenericParser::error): Overloaded method to handle the case
791         of the location being passed in too. Ideally, this should go later since all
792         error reporting should be done through the Report object.
793
794         * class.cs (TypeContainer.Indexers): New property to access the list of indexers.
795         (Populate): Iterate thru the indexers we have and define them too.
796         (Indexer.GetMethodBuilder, .SetMethodBuilder): New members to hold the method builders
797         for the get and set accessors.
798         (Indexer::Define): Implement.
799         
800 2001-09-09  Miguel de Icaza  <miguel@ximian.com>
801
802         * expression.cs (Binary::Resolve): Beginning of it.  I scratched
803         my previous implementation, did not work.
804
805         * typemanager.cs: Add a couple of missing types (the longs).
806
807         * literal.cs: Use TypeManager.bool_type instead of getting it.
808
809         * expression.cs (EventExpr): New kind of expressions.
810         (Expressio::ExprClassFromMemberInfo): finish
811
812 2001-09-08  Miguel de Icaza  <miguel@ximian.com>
813
814         * assign.cs: Emit stores to static fields differently.
815
816 2001-09-08  Ravi Pratap  <ravi@ximian.com>
817
818         * Merge in changes and adjust code to tackle conflicts. Backed out my
819         code in Assign::Resolve ;-) 
820
821 2001-09-08  Ravi Pratap  <ravi@ximian.com>
822
823         * cs-parser.jay (CheckAttributeTarget): Modify call to error to use
824         instead Report.Error and also pass in the location.
825         (CSharpParser::Lexer): New readonly property to return the reference
826         to the Tokenizer object.
827         (declare_local_variables): Use Report.Error with location instead of plain 
828         old error.
829         (CheckDef): Ditto.
830
831         * class.cs (Operator::CheckUnaryOperator): Move into cs-parser.jay.
832         (Operator.CheckBinaryOperator): Ditto.
833
834         * cs-parser.jay (operator_declarator): Update accordingly.
835
836         * cs-parser.jay (CheckUnaryOperator): Modify to use Report.Error
837         (CheckBinaryOperator): Same here.
838
839         * rootcontext.cs (LookupType): Add an extra lookup which simply does a lookup
840         on the name without any prefixes of namespace names etc. This is because we
841         already might have something already fully qualified like 
842         'System.Console.WriteLine'
843
844         * assign.cs (Resolve): Begin implementation. Stuck ;-)
845
846 2001-09-07  Ravi Pratap  <ravi@ximian.com>
847
848         * cs-tokenizer.cs (location): Return a string which also contains
849         the file name.
850
851         * expression.cs (ElementAccess): New class for expressions of the
852         type 'element access.'
853         (BaseAccess): New class for expressions of the type 'base access.'
854         (CheckedExpr, UnCheckedExpr): New classes for Checked and Unchecked expressions
855         respectively.
856         
857         * cs-parser.jay (element_access): Implement action.
858         (base_access): Implement actions.
859         (checked_expression, unchecked_expression): Implement.
860
861         * cs-parser.jay (local_variable_type): Correct and implement.
862         (type_suffixes, type_suffix_list, type_suffix): Implement actions.
863
864         * cs-tokenizer.cs (real_type_suffix): Comment out the extra getchar.
865
866         * cs-parser.jay (rank_specifiers): Remove space while concatenating the type's
867         name and the specifiers.
868
869         * interface.cs (InterfaceAttr): New property to return the corresponding TypeAttributes
870         
871         * rootcontext.cs (CreateInterface): Use the InterfaceAttr property instead of 
872         making them all public ;-)
873
874         * cs-parser.jay (error): Remove entirely as we have an implementation in the base
875         class anyways.
876         
877 2001-09-07  Miguel de Icaza  <miguel@ximian.com>
878
879         * expression.cs (ExprClassFromMemberInfo): Return FieldExpr and
880         PropertyExprs.
881         (FieldExpr, PropertyExprs): New resolved expressions.
882         (SimpleName::MemberStaticCheck): Perform static checks for access
883         to non-static fields on static methods. Maybe this should be
884         generalized for MemberAccesses. 
885         (SimpleName::ResolveSimpleName): More work on simple name
886         resolution. 
887
888         * cs-parser.jay (primary_expression/qualified_identifier): track
889         the parameter index.
890
891         * codegen.cs (CodeGen::Save): Catch save exception, report error.
892         (EmitContext::EmitBoolExpression): Chain to expression generation
893         instead of temporary hack.
894         (::EmitStatementExpression): Put generic expression code generation.
895
896         * assign.cs (Assign::Emit): Implement variable assignments to
897         local variables, parameters and fields.
898
899 2001-09-06  Miguel de Icaza  <miguel@ximian.com>
900
901         * statement.cs (Block::GetVariableInfo): New method, returns the
902         VariableInfo for a variable name in a block.
903         (Block::GetVariableType): Implement in terms of GetVariableInfo
904
905         * literal.cs (IntLiteral::Emit, FloatLiteral::Emit,
906         DoubleLiteral::Emit, CharLiteral::Emit, BoolLiteral::Emit): Implement
907
908 2001-09-06  Ravi Pratap  <ravi@ximian.com>
909
910         * cs-parser.jay (operator_declaration): Continue on my quest : update
911         to take attributes argument.
912         (event_declaration): Ditto.
913         (enum_declaration): Ditto.
914         (indexer_declaration): Ditto.
915         
916         * class.cs (Operator::Operator): Update constructor accordingly.
917         (Event::Event): Ditto.
918
919         * delegate.cs (Delegate::Delegate): Same here.
920
921         * enum.cs (Enum::Enum): Same here.
922         
923 2001-09-05  Ravi Pratap  <ravi@ximian.com>
924
925         * cs-parser.jay (CheckAttributeTarget): Update to use the right error number.
926
927         * ../tests/cs0658.cs : New file to demonstrate error 0658.
928
929         * attribute.cs (Attributes): New class to encapsulate all attributes which were
930         being passed around as an arraylist.
931         (Attributes::AddAttribute): Method to add attribute sections.
932
933         * cs-parser.jay (opt_attributes): Modify actions to use the new Attributes class.
934         (struct_declaration): Update accordingly.
935         (constant_declaration): Update.
936         (field_declaration): Update.
937         (method_header): Update.
938         (fixed_parameter): Update.
939         (parameter_array): Ditto.
940         (property_declaration): Ditto.
941         (destructor_declaration): Ditto.
942         
943         * class.cs (Struct::Struct): Update constructors accordingly.
944         (Class::Class): Ditto.
945         (Field::Field): Ditto.
946         (Method::Method): Ditto.
947         (Property::Property): Ditto.
948         (TypeContainer::OptAttribute): update property's return type.
949         
950         * interface.cs (Interface.opt_attributes): New member.
951         (Interface::Interface): Update to take the extra Attributes argument.
952
953         * parameter.cs (Parameter::Parameter): Ditto.
954
955         * constant.cs (Constant::Constant): Ditto.
956
957         * interface.cs (InterfaceMemberBase): New OptAttributes field.
958         (InterfaceMemberBase::InterfaceMemberBase): Update constructor to take 
959         the attributes as a parameter.
960         (InterfaceProperty): Update constructor call.
961         (InterfaceEvent): Ditto.
962         (InterfaceMethod): Ditto.
963         (InterfaceIndexer): Ditto.
964
965         * cs-parser.jay (interface_indexer_declaration): Update call to constructor to 
966         pass the attributes too.
967         (interface_event_declaration): Ditto.
968         (interface_property_declaration): Ditto.
969         (interface_method_declaration): Ditto.
970         (interface_declaration): Ditto.
971
972 2001-09-05  Miguel de Icaza  <miguel@ximian.com>
973
974         * class.cs (Method::Define): Track the "static Main" definition to
975         create an entry point. 
976
977         * rootcontext.cs (RootContext::EntryPoint): MethodInfo that holds the
978         EntryPoint if we find it. 
979
980         * codegen.cs (EmitContext::EmitInvocation): Emit invocations.
981         (EmitContext::ig): Make this variable public.
982
983         * driver.cs: Make the default output file be the first file name
984         with the .exe extension.  
985
986         Detect empty compilations
987
988         Handle various kinds of output targets.  Handle --target and
989         rename -t to --dumper.
990
991         * expression.cs, literal.cs, assign.cs, constant.cs: All `Resolve'
992         methods inherited from Expression return now an Expression.  This
993         will is used during the tree rewriting as we resolve them during
994         semantic analysis.
995
996         (Expression::MemberLookup): Implements the MemberLookup (7.3) from
997         the spec.  Missing entirely is the information about
998         accessability of elements of it.
999
1000         (Expression::ExprClassFromMemberInfo): New constructor for
1001         Expressions that creates a fully initialized Expression based on
1002         a MemberInfo that is one of Eventinfo, FieldINfo, PropertyInfo or
1003         a Type.
1004
1005         (Invocation::Resolve): Begin implementing resolution of invocations.
1006         
1007         * literal.cs (StringLiteral):  Implement Emit.
1008
1009 2001-09-05  Ravi Pratap  <ravi@ximian.com>
1010
1011         * cs-parser.jay (error): Add new modifier because we are hiding an inherited
1012         member.
1013         
1014 2001-09-04  Ravi Pratap  <ravi@ximian.com>
1015
1016         * cs-parser.jay (attribute_arguments): Implement actions.
1017         (attribute): Fix bug in production. Implement action.
1018         (attribute_list): Implement.
1019         (attribute_target): Implement.
1020         (attribute_target_specifier, opt_target_specifier): Implement
1021         (CheckAttributeTarget): New method to check if the attribute target
1022         is valid.
1023         (attribute_section): Implement.
1024         (opt_attributes): Implement.
1025
1026         * attribute.cs : New file to handle attributes.
1027         (Attribute): Class to hold attribute info.
1028
1029         * cs-parser.jay (opt_attribute_target_specifier): Remove production
1030         (attribute_section): Modify production to use 2 different rules to 
1031         achieve the same thing. 1 s/r conflict down !
1032         Clean out commented, useless, non-reducing dimension_separator rules.
1033         
1034         * class.cs (TypeContainer.attributes): New member to hold list
1035         of attributes for a type.
1036         (Struct::Struct): Modify to take one more argument, the attribute list.
1037         (Class::Class): Ditto.
1038         (Field::Field): Ditto.
1039         (Method::Method): Ditto.
1040         (Property::Property): Ditto.
1041         
1042         * cs-parser.jay (struct_declaration): Update constructor call to
1043         pass in the attributes too.
1044         (class_declaration): Ditto.
1045         (constant_declaration): Ditto.
1046         (field_declaration): Ditto.
1047         (method_header): Ditto.
1048         (fixed_parameter): Ditto.
1049         (parameter_array): Ditto.
1050         (property_declaration): Ditto.
1051
1052         * constant.cs (Constant::Constant): Update constructor similarly.
1053         Use System.Collections.
1054
1055         * parameter.cs (Parameter::Parameter): Update as above.
1056
1057 2001-09-02  Ravi Pratap  <ravi@ximian.com>
1058
1059         * class.cs (TypeContainer::AddDelegate): New method to add a delegate.
1060         (TypeContainer.delegates): New member to hold list of delegates.
1061
1062         * cs-parser.jay (delegate_declaration): Implement the action correctly 
1063         this time as I seem to be on crack ;-)
1064
1065 2001-09-02  Miguel de Icaza  <miguel@ximian.com>
1066
1067         * rootcontext.cs (RootContext::IsNamespace): new function, used to
1068         tell whether an identifier represents a namespace.
1069
1070         * expression.cs (NamespaceExpr): A namespace expression, used only
1071         temporarly during expression resolution.
1072         (Expression::ResolveSimpleName, ::ResolvePrimary, ::ResolveName):
1073         utility functions to resolve names on expressions.
1074
1075 2001-09-01  Miguel de Icaza  <miguel@ximian.com>
1076
1077         * codegen.cs: Add hook for StatementExpressions. 
1078
1079         * class.cs: Fix inverted test for static flag in methods.
1080
1081 2001-09-02  Ravi Pratap  <ravi@ximian.com>
1082
1083         * class.cs (Operator::CheckUnaryOperator): Correct error number used
1084         to make it coincide with MS' number.
1085         (Operator::CheckBinaryOperator): Ditto.
1086
1087         * ../errors/errors.txt : Remove error numbers added earlier.
1088
1089         * ../errors/cs1019.cs : Test case for error # 1019
1090
1091         * ../errros/cs1020.cs : Test case for error # 1020
1092
1093         * cs-parser.jay : Clean out commented cruft.
1094         (dimension_separators, dimension_separator): Comment out. Ostensibly not
1095         used anywhere - non-reducing rule.
1096         (namespace_declarations): Non-reducing rule - comment out.
1097
1098         * enum.cs (Enum::AddEnum): Rename to AddEnumMember as I was getting confused
1099         with TypeContainer::AddEnum.
1100
1101         * delegate.cs : New file for delegate handling classes.
1102         (Delegate): Class for declaring delegates.
1103
1104         * makefile : Update.
1105
1106         * cs-parser.jay (delegate_declaration): Implement.
1107
1108 2001-09-01  Ravi Pratap  <ravi@che.iitm.ac.in>
1109
1110         * class.cs (Event::Define): Implement.
1111         (Event.EventBuilder): New member.
1112
1113         * class.cs (TypeContainer::Populate): Update to define all enums and events
1114         we have.
1115         (Events): New property for the events arraylist we hold. Shouldn't we move to using
1116         readonly fields for all these cases ?
1117
1118 2001-08-31  Ravi Pratap  <ravi@che.iitm.ac.in>
1119
1120         * class.cs (Property): Revamp to use the convention of making fields readonly.
1121         Accordingly modify code elsewhere.
1122
1123         * class.cs : Apply patch from Mr. Mandar <go_mono@hotmail.com> for implementing
1124         the Define method of the Property class.
1125
1126         * class.cs : Clean up applied patch and update references to variables etc. Fix 
1127         trivial bug.
1128         (TypeContainer::Populate): Update to define all the properties we have. Also
1129         define all enumerations.
1130
1131         * enum.cs (Define): Implement.
1132         
1133 2001-08-31  Ravi Pratap  <ravi@che.iitm.ac.in>
1134
1135         * cs-parser.jay (overloadable_operator): The semantic value is an
1136         enum of the Operator class.
1137         (operator_declarator): Implement actions.
1138         (operator_declaration): Implement.
1139
1140         * class.cs (Operator::CheckUnaryOperator): New static method to help in checking
1141         validity of definitions.
1142         (Operator::CheckBinaryOperator): Static method to check for binary operators
1143         (TypeContainer::AddOperator): New method to add an operator to a type.
1144
1145         * cs-parser.jay (indexer_declaration): Added line to actually call the
1146         AddIndexer method so it gets added ;-)
1147
1148         * ../errors/errors.txt : Update to include new error numbers. Are these numbers 
1149         already taken care of by the MS compiler ?  
1150
1151 2001-08-29  Ravi Pratap  <ravi@che.iitm.ac.in>
1152
1153         * class.cs (Operator): New class for operator declarations.
1154         (Operator::OpType): Enum for the various operators.
1155
1156 2001-08-29  Ravi Pratap  <ravi@che.iitm.ac.in>
1157
1158         * class.cs (TypeContainer::AddIndexer): Remove FIXME comment. We
1159         ostensibly handle this in semantic analysis.
1160
1161         * cs-parser.jay (general_catch_clause): Comment out
1162         (specific_catch_clauses, specific_catch_clause): Ditto.
1163         (opt_general_catch_clause, opt_specific_catch_clauses): Ditto
1164         (catch_args, opt_catch_args): New productions.
1165         (catch_clause): Rewrite to use the new productions above
1166         (catch_clauses): Modify accordingly.
1167         (opt_catch_clauses): New production to use in try_statement
1168         (try_statement): Revamp. Basically, we get rid of one unnecessary rule
1169         and re-write the code in the actions to extract the specific and
1170         general catch clauses by being a little smart ;-)
1171
1172         * ../tests/try.cs : Fix. It's not 'finalize' my friend, it's 'finally' !
1173         Hooray, try and catch statements parse fine !
1174         
1175 2001-08-28  Ravi Pratap  <ravi@che.iitm.ac.in>
1176
1177         * statement.cs (Block::GetVariableType): Fix logic to extract the type
1178         string from the hashtable of variables.
1179
1180         * cs-parser.jay (event_accessor_declarations): Trivial fix. Man, how did
1181         I end up making that mistake ;-)
1182         (catch_clauses): Fixed gross error which made Key and Value of the 
1183         DictionaryEntry the same : $1 !!
1184
1185 2001-08-28  Ravi Pratap  <ravi@che.iitm.ac.in>
1186
1187         * cs-tokenizer.cs (initTokens): Add keywords 'add' and 'remove'
1188
1189         * cs-parser.jay (event_declaration): Correct to remove the semicolon
1190         when the add and remove accessors are specified. 
1191
1192 2001-08-28  Ravi Pratap  <ravi@che.iitm.ac.in>
1193
1194         * cs-parser.jay (IndexerDeclaration): New helper class to hold
1195         information about indexer_declarator.
1196         (indexer_declarator): Implement actions.
1197         (parsing_indexer): New local boolean used to keep track of whether
1198         we are parsing indexers or properties. This is necessary because 
1199         implicit_parameters come into picture even for the get accessor in the 
1200         case of an indexer.
1201         (get_accessor_declaration, set_accessor_declaration): Correspondingly modified.
1202
1203         * class.cs (Indexer): New class for indexer declarations.
1204         (TypeContainer::AddIndexer): New method to add an indexer to a type.
1205         (TypeContainer::indexers): New member to hold list of indexers for the
1206         type.
1207
1208 2001-08-27  Ravi Pratap  <ravi@che.iitm.ac.in>
1209
1210         * cs-parser.jay (add_accessor_declaration): Implement action.
1211         (remove_accessor_declaration): Implement action.
1212         (event_accessors_declaration): Implement
1213         (variable_declarators): swap statements for first rule - trivial.
1214
1215         * class.cs (Event): New class to hold information about event
1216         declarations.
1217         (TypeContainer::AddEvent): New method to add an event to a type
1218         (TypeContainer::events): New member to hold list of events.
1219
1220         * cs-parser.jay (event_declaration): Implement actions.
1221
1222 2001-08-27  Ravi Pratap  <ravi@che.iitm.ac.in>
1223
1224         * cs-parser.jay (dim_separators): Implement. Make it a string
1225         concatenating all the commas together, just as they appear.
1226         (opt_dim_separators): Modify accordingly
1227         (rank_specifiers): Update accordingly. Basically do the same
1228         thing - instead, collect the brackets here.
1229         (opt_rank_sepcifiers): Modify accordingly.
1230         (array_type): Modify to actually return the complete type string
1231         instead of ignoring the rank_specifiers.
1232         (expression_list): Implement to collect the expressions
1233         (variable_initializer): Implement. We make it a list of expressions
1234         essentially so that we can handle the array_initializer case neatly too.
1235         (variable_initializer_list): Implement.
1236         (array_initializer): Make it a list of variable_initializers
1237         (opt_array_initializer): Modify accordingly.
1238
1239         * expression.cs (New::NType): Add enumeration to help us
1240         keep track of whether we have an object/delegate creation
1241         or an array creation.
1242         (New:NewType, New::Rank, New::Indices, New::Initializers): New
1243         members to hold data about array creation.
1244         (New:New): Modify to update NewType
1245         (New:New): New Overloaded contructor for the array creation
1246         case.
1247
1248         * cs-parser.jay (array_creation_expression): Implement to call
1249         the overloaded New constructor.
1250         
1251 2001-08-26  Ravi Pratap  <ravi@che.iitm.ac.in>
1252
1253         * class.cs (TypeContainer::Constructors): Return member
1254         constructors instead of returning null.
1255
1256 2001-08-26  Miguel de Icaza  <miguel@ximian.com>
1257
1258         * typemanager.cs (InitCoreTypes): Initialize the various core
1259         types after we have populated the type manager with the user
1260         defined types (this distinction will be important later while
1261         compiling corlib.dll)
1262
1263         * expression.cs, literal.cs, assign.cs, constant.cs: Started work
1264         on Expression Classification.  Now all expressions have a method
1265         `Resolve' and a method `Emit'.
1266
1267         * codegen.cs, cs-parser.jay: Fixed the bug that stopped code
1268         generation from working.     Also add some temporary debugging
1269         code. 
1270         
1271 2001-08-24  Miguel de Icaza  <miguel@ximian.com>
1272
1273         * codegen.cs: Lots of code generation pieces.  This is only the
1274         beginning, will continue tomorrow with more touches of polish.  We
1275         handle the fundamentals of if, while, do, for, return.  Others are
1276         trickier and I need to start working on invocations soon.
1277         
1278         * gen-treedump.cs: Bug fix, use s.Increment here instead of
1279         s.InitStatement. 
1280
1281         * codegen.cs (EmitContext): New struct, used during code
1282         emission to keep a context.   Most of the code generation will be
1283         here. 
1284
1285         * cs-parser.jay: Add embedded blocks to the list of statements of
1286         this block.  So code generation proceeds in a top down fashion.
1287
1288 2001-08-23  Miguel de Icaza  <miguel@ximian.com>
1289
1290         * statement.cs: Add support for multiple child blocks.
1291
1292 2001-08-22  Miguel de Icaza  <miguel@ximian.com>
1293
1294         * codegen.cs (EmitCode): New function, will emit the code for a
1295         Block of code given a TypeContainer and its ILGenerator. 
1296
1297         * statement.cs (Block): Standard public readonly optimization.
1298         (Block::Block constructors): Link children. 
1299         (Block::Child): Child Linker.
1300         (Block::EmitVariables): Emits IL variable declarations.
1301
1302         * class.cs: Drop support for MethodGroups here, delay until
1303         Semantic Analysis.
1304         (Method::): Applied the same simplification that I did before, and
1305         move from Properties to public readonly fields.
1306         (Method::ParameterTypes): Returns the parameter types for the
1307         function, and implements a cache that will be useful later when I
1308         do error checking and the semantic analysis on the methods is
1309         performed.
1310         (Constructor::GetCallingConvention): Renamed from CallingConvetion
1311         and made a method, optional argument tells whether this is a class
1312         or a structure to apply the `has-this' bit.
1313         (Method::GetCallingConvention): Implement, returns the calling
1314         convention. 
1315         (Method::Define): Defines the type, a second pass is performed
1316         later to populate the methods.
1317
1318         (Constructor::ParameterTypes): implement a cache similar to the
1319         one on Method::ParameterTypes, useful later when we do semantic
1320         analysis. 
1321
1322         (TypeContainer::EmitMethod):  New method.  Emits methods.
1323
1324         * expression.cs: Removed MethodGroup class from here.
1325         
1326         * parameter.cs (Parameters::GetCallingConvention): new method.
1327
1328 2001-08-21  Miguel de Icaza  <miguel@ximian.com>
1329
1330         * class.cs (TypeContainer::Populate): Drop RootContext from the
1331         argument. 
1332
1333         (Constructor::CallingConvention): Returns the calling convention.
1334         (Constructor::ParameterTypes): Returns the constructor parameter
1335         types. 
1336         
1337         (TypeContainer::AddConstructor): Keep track of default constructor
1338         and the default static constructor.
1339
1340         (Constructor::) Another class that starts using `public readonly'
1341         instead of properties. 
1342
1343         (Constructor::IsDefault): Whether this is a default constructor. 
1344
1345         (Field::) use readonly public fields instead of properties also.
1346
1347         (TypeContainer::TypeAttr, TypeContainer::AddConstructor): Keep
1348         track of static constructors;  If none is used, turn on
1349         BeforeFieldInit in the TypeAttributes. 
1350
1351         * cs-parser.jay (opt_argument_list): now the return can be null
1352         for the cases where there are no arguments. 
1353
1354         (constructor_declarator): If there is no implicit `base' or
1355         `this', then invoke the default parent constructor. 
1356         
1357         * modifiers.cs (MethodAttr): New static function maps a set of
1358         modifiers flags into a MethodAttributes enum
1359         (FieldAttr): renamed from `Map'.  So now we have FieldAttr,
1360         MethodAttr, TypeAttr to represent the various mappings where the
1361         modifiers are used.
1362         (FieldAttr): Map also `readonly' to `FieldAttributes.InitOnly'  
1363
1364 2001-08-19  Miguel de Icaza  <miguel@ximian.com>
1365
1366         * parameter.cs (GetParameterInfo): Fix bug where there would be no
1367         method arguments.
1368
1369         * interface.cs (PopulateIndexer): Implemented the code generator
1370         for interface indexers.
1371
1372 2001-08-17  Miguel de Icaza  <miguel@ximian.com>
1373
1374         * interface.cs (InterfaceMemberBase): Now we track the new status
1375         here.  
1376
1377         (PopulateProperty): Implement property population.  Woohoo!  Got
1378         Methods and Properties going today. 
1379
1380         Removed all the properties for interfaces, and replaced them with
1381         `public readonly' fields. 
1382
1383 2001-08-16  Miguel de Icaza  <miguel@ximian.com>
1384
1385         * interface.cs (AddEvent, AddMethod, AddIndexer, AddProperty):
1386         initialize their hashtables/arraylists only when they are needed
1387         instead of doing this always.
1388
1389         * parameter.cs: Handle refs and out parameters.
1390
1391         * cs-parser.jay: Use an ArrayList to construct the arguments
1392         instead of the ParameterCollection, and then cast that to a
1393         Parameter[] array.
1394
1395         * parameter.cs: Drop the use of ParameterCollection and use
1396         instead arrays of Parameters.
1397
1398         (GetParameterInfo): Use the Type, not the Name when resolving
1399         types. 
1400
1401 2001-08-13  Miguel de Icaza  <miguel@ximian.com>
1402
1403         * parameter.cs: Eliminate the properties Name, Type and ModFlags,
1404         and instead use public readonly fields.
1405
1406         * class.cs: Put back walking code for type containers.
1407
1408 2001-08-11  Miguel de Icaza  <miguel@ximian.com>
1409
1410         * class.cs (MakeConstant): Code to define constants.
1411
1412         * rootcontext.cs (LookupType): New function.  Used to locate types 
1413
1414         
1415 2001-08-08  Miguel de Icaza  <miguel@ximian.com>
1416
1417         * rootcontext.cs: OH MY!  My trick works!   It is amazing how nice
1418         this System.Reflection code is.  Kudos to Microsoft
1419         
1420         * typemanager.cs: Implement a type cache and avoid loading all
1421         types at boot time.  Wrap in LookupType the internals.  This made
1422         the compiler so much faster.  Wow.  I rule!
1423         
1424         * driver.cs: Make sure we always load mscorlib first (for
1425         debugging purposes, nothing really important).
1426
1427         * Renamespaced things that were on `CSC' to `CIR'.  Maybe I should
1428         have moved to `CSC' rather than `CIR'.  Oh man!  The confussion!  
1429
1430         * rootcontext.cs: Lookup types on their namespace;  Lookup types
1431         on namespaces that have been imported using the `using' keyword.
1432
1433         * class.cs (TypeContainer::TypeAttr): Virtualize.
1434         (Class::TypeAttr): Return attributes suitable for this bad boy.
1435         (Struct::TypeAttr): ditto.
1436         Handle nested classes.
1437         (TypeContainer::) Remove all the type visiting code, it is now
1438         replaced with the rootcontext.cs code
1439
1440         * rootcontext.cs (GetClassBases): Added support for structs. 
1441
1442 2001-08-06  Miguel de Icaza  <miguel@ximian.com>
1443
1444         * interface.cs, statement.cs, class.cs, parameter.cs,
1445         rootcontext.cs, gen-treedump.cs, enum.cs, cs-parse.jay:
1446         Drop use of TypeRefs, and use strings instead.
1447
1448 2001-08-04  Miguel de Icaza  <miguel@ximian.com>
1449
1450         * rootcontext.cs: 
1451
1452         * class.cs (Struct::Struct): set the SEALED flags after
1453         checking the modifiers.
1454         (TypeContainer::TypeAttr): new property, returns the
1455         TypeAttributes for a class.  
1456
1457         * cs-parser.jay (type_list): Oops, list production was creating a
1458         new list of base types.
1459
1460         * rootcontext.cs (StdLib): New property.
1461         (GetInterfaceTypeByName): returns an interface by type name, and
1462         encapsulates error handling here.
1463         (GetInterfaces): simplified.
1464         (ResolveTree): Encapsulated all the tree resolution here.
1465         (CreateClass, GetClassBases, GetInterfaceOrClass): Create class
1466         types. 
1467         
1468         * driver.cs: Add support for --nostdlib, to avoid loading the
1469         default assemblies.
1470         (Main): Do not put tree resolution here. 
1471
1472         * rootcontext.cs: Beginning of the class resolution.
1473
1474 2001-08-03  Miguel de Icaza  <miguel@ximian.com>
1475
1476         * rootcontext.cs: Provide better error reporting. 
1477
1478         * cs-parser.jay (interface_base): set our $$ to be interfaces.
1479
1480         * rootcontext.cs (CreateInterface): Handle the case where there
1481         are no parent interfaces.
1482         
1483         (CloseTypes): Routine to flush types at the end.
1484         (CreateInterface): Track types.
1485         (GetInterfaces): Returns an array of Types from the list of
1486         defined interfaces.
1487
1488         * typemanager.c (AddUserType): Mechanism to track user types (puts
1489         the type on the global type hash, and allows us to close it at the
1490         end). 
1491         
1492 2001-08-02  Miguel de Icaza  <miguel@ximian.com>
1493
1494         * tree.cs: Removed RecordType, added RecordClass, RecordStruct and
1495         RecordInterface instead.
1496
1497         * cs-parser.jay: Updated to reflect changes above.
1498
1499         * decl.cs (Definition): Keep track of the TypeBuilder type that
1500         represents this type here.  Not sure we will use it in the long
1501         run, but wont hurt for now.
1502
1503         * driver.cs: Smaller changes to accomodate the new code.
1504
1505         Call ResolveInterfaceBases, Call ResolveClassBases, Save assembly
1506         when done. 
1507
1508         * rootcontext.cs (CreateInterface):  New method, used to create
1509         the System.TypeBuilder type for interfaces.
1510         (ResolveInterfaces): new entry point to resolve the interface
1511         hierarchy. 
1512         (CodeGen): Property, used to keep track of the code generator.
1513
1514 2001-07-26  Miguel de Icaza  <miguel@ximian.com>
1515
1516         * cs-parser.jay: Add a second production for delegate_declaration
1517         with `VOID'.
1518
1519         (enum_body): Put an opt_comma here instead of putting it on
1520         enum_body or enum_member_declarations so we can handle trailing
1521         commas on enumeration members.  Gets rid of a shift/reduce.
1522         
1523         (type_list): Need a COMMA in the middle.
1524
1525         (indexer_declaration): Tell tokenizer to recognize get/set
1526
1527         * Remove old targets.
1528
1529         * Re-add the parser target.
1530
1531 2001-07-13  Simon Cozens <simon@simon-cozens.org>
1532
1533         * cs-parser.jay: Add precendence rules for a number of operators
1534         ot reduce the number of shift/reduce conflicts in the grammar.
1535         
1536 2001-07-17  Miguel de Icaza  <miguel@ximian.com>
1537
1538         * tree.cs: moved IGenerator interface and renamed it to ITreeDump
1539         and put it here.
1540
1541         Get rid of old crufty code.
1542
1543         * rootcontext.cs: Use this to keep track of the parsed
1544         representation and the defined types available to the program. 
1545
1546         * gen-treedump.cs: adjust for new convention.
1547
1548         * type.cs: Split out the type manager, and the assembly builder
1549         from here. 
1550
1551         * typemanager.cs: the type manager will live here now.
1552
1553         * cil-codegen.cs: And the code generator here. 
1554
1555 2001-07-14  Sean MacIsaac  <macisaac@ximian.com>
1556
1557         * makefile: Fixed up for easy making.
1558
1559 2001-07-13  Simon Cozens <simon@simon-cozens.org>
1560
1561         * cs-parser.jay (rank_specifier): Remove a conflict by reordering
1562         the 
1563
1564         (unary_expression): Expand pre_increment_expression and
1565         post_decrement_expression to reduce a shift/reduce.
1566
1567 2001-07-11  Simon Cozens
1568
1569         * cs-tokenizer.cs: Hex numbers should begin with a 0.
1570
1571         Improve allow_keyword_as_indent name.
1572
1573 2001-06-19  Miguel de Icaza  <miguel@ximian.com>
1574
1575         * Adjustments for Beta2. 
1576
1577 2001-06-13  Miguel de Icaza  <miguel@ximian.com>
1578
1579         * decl.cs: Added `Define' abstract method.
1580         (InTransit): new property, used to catch recursive definitions. 
1581
1582         * interface.cs: Implement `Define'. 
1583
1584         * modifiers.cs: Map Modifiers.constants to
1585         System.Reflection.TypeAttribute flags.
1586
1587         * class.cs: Keep track of types and user-defined types.
1588         (BuilderInit): New method for creating an assembly
1589         (ResolveType): New function to launch the resolution process, only
1590         used by interfaces for now.
1591
1592         * cs-parser.jay: Keep track of Classes, Structs and Interfaces
1593         that are inserted into the name space. 
1594
1595 2001-06-08  Miguel de Icaza  <miguel@ximian.com>
1596
1597         * ARGH.  I have screwed up my tree so many times due to the use of
1598         rsync rather than using CVS.  Going to fix this at once. 
1599
1600         * driver.cs: Objetify driver.  Load assemblies, use assemblies to
1601         load types.
1602
1603 2001-06-07  Miguel de Icaza  <miguel@ximian.com>
1604
1605         * Experiment successful: Use System.Type rather that our own
1606         version of Type.  
1607
1608 2001-05-25  Miguel de Icaza  <miguel@ximian.com>
1609
1610         * cs-parser.jay: Removed nsAliases from here.
1611
1612         Use new namespaces, handle `using XXX;' 
1613
1614         * namespace.cs: Reimplemented namespace handling, use a recursive
1615         definition of the class.  Now we can keep track of using clauses
1616         and catch invalid using clauses.
1617
1618 2001-05-24  Miguel de Icaza  <miguel@ximian.com>
1619
1620         * gen-treedump.cs: Adapted for all the renaming.
1621
1622         * expression.cs (Expression): this class now has a Type property
1623         which returns an expression Type.
1624
1625         (Probe::, New::, TypeOf::, SizeOf::, Constant::): renamed from
1626         `Type', as this has a different meaning now in the base
1627
1628 2001-05-22  Miguel de Icaza  <miguel@ximian.com>
1629
1630         * interface.cs, class.cs: Removed from all the sources the
1631         references to signature computation, as we can not do method
1632         signature computation during the parsing time, as we are not
1633         trying to solve at that point distinguishing:
1634
1635         class X {
1636                 void a (Blah x) {}
1637                 void a (NS.Blah x) {}
1638         }
1639
1640         Which depending on the context might be valid or not, as we do not
1641         know if Blah is the same thing as NS.Blah at that point.
1642
1643         * Redid everything so the code uses TypeRefs now instead of
1644         Types.  TypeRefs are just temporary type placeholders, that need
1645         to be resolved.  They initially have a pointer to a string and the
1646         current scope in which they are used.  This is used later by the
1647         compiler to resolve the reference to an actual Type. 
1648
1649         * DeclSpace is no longer a CIR.Type, and neither are
1650         TypeContainers (Class and Struct) nor Interfaces nor Enums.  They
1651         are all DeclSpaces, but no Types. 
1652
1653         * type.cs (TypeRefManager): This implements the TypeRef manager,
1654         which keeps track of all the types that need to be resolved after
1655         the parsing has finished. 
1656
1657 2001-05-13  Miguel de Icaza  <miguel@ximian.com>
1658
1659         * ARGH.  We are going to have to store `foreach' as a class rather
1660         than resolving it, as we need to verify error 1579 after name
1661         resolution.   *OR* we could keep a flag that says `This request to
1662         IEnumerator comes from a foreach statement' which we can then use
1663         to generate the error.
1664
1665 2001-05-10  Miguel de Icaza  <miguel@ximian.com>
1666
1667         * class.cs (TypeContainer.AddMethod): we now add methods to the
1668         MethodGroup instead of the method hashtable.  
1669
1670         * expression.cs: Add MethodGroup abstraction, which gets us one
1671         step closer to the specification in the way we handle method
1672         declarations.  
1673
1674         * cs-parser.jay (primary_expression): qualified_identifier now
1675         tried to match up an identifier to a local variable reference or
1676         to a parameter reference.
1677
1678         current_local_parameters is now a parser global variable that
1679         points to the current parameters for the block, used during name
1680         lookup.
1681
1682         (property_declaration): Now creates an implicit `value' argument to
1683         the set accessor.
1684
1685 2001-05-09  Miguel de Icaza  <miguel@ximian.com>
1686
1687         * parameter.cs: Do not use `param' arguments as part of the
1688         signature, per the spec.
1689
1690 2001-05-08  Miguel de Icaza  <miguel@ximian.com>
1691
1692         * decl.cs: Base class for classes, structs and interfaces.  This
1693         is the "Declaration Space" 
1694
1695         * cs-parser.jay: Use CheckDef for checking declaration errors
1696         instead of having one on each function.
1697
1698         * class.cs: Factor out some code for handling error handling in
1699         accordance to the "Declarations" section in the "Basic Concepts"
1700         chapter in the ECMA C# spec.
1701
1702         * interface.cs: Make all interface member classes derive from
1703         InterfaceMemberBase.
1704
1705 2001-05-07  Miguel de Icaza  <miguel@ximian.com>
1706
1707         * Many things: all interfaces are parsed and generated in
1708         gen-treedump.  Support for member variables, constructors,
1709         destructors, properties, constants is there.
1710
1711         Beginning of the IL backend, but very little done, just there for
1712         testing purposes. 
1713
1714 2001-04-29  Miguel de Icaza  <miguel@ximian.com>
1715
1716         * cs-parser.jay: Fix labeled statement.
1717
1718         * cs-tokenizer.cs (escape): Escape " and ' always.
1719         ref_line, ref_name: keep track of the line/filename as instructed
1720         by #line by the compiler.
1721         Parse #line.
1722
1723 2001-04-27  Miguel de Icaza  <miguel@ximian.com>
1724
1725         * System.CodeDOM/CodeBinaryOperatorExpression.cs: Rearrange enum
1726         to match the values in System.CodeDOM.
1727
1728         Divid renamed to Divide.
1729
1730         * System.CodeDOM/CodeForLoopStatement.cs: Always have valid
1731         statements. 
1732         (Statements.set): remove.
1733
1734         * System.CodeDOM/CodeCatchClause.cs: always have a valid
1735         statements. 
1736
1737         * System.CodeDOM/CodeIfStatement.cs: trueStatements and
1738         falseStatements always have valid values. 
1739
1740         * cs-parser.jay: Use System.CodeDOM now.
1741