Fixes for compiling mPhoto:
[mono.git] / mcs / mcs / ChangeLog
index 484a3039dd4c4124c399c79e56f39255c47bbf2a..fd30a8df217d94498cece1c948f4e868aab0f6eb 100755 (executable)
@@ -1,6 +1,202 @@
+2003-05-12  Miguel de Icaza  <miguel@ximian.com>
+
+       * support.cs (InternalParameters.ParameterType): Return the
+       ExternalType of the parameter.
+
+       * parameter.cs (Parameter.ExternalType): drop the two arguments,
+       they were unused.
+
+s2003-05-11  Miguel de Icaza  <miguel@ximian.com>
+
+       * class.cs (MethodData.Define): Do not set the `newslot' on
+       interface members, if they are also flagged as "override".
+
+       * expression.cs (UnaryMutator.EmitCode): Simple workaround to emit
+       better code for ++i and i++.  This only works for static fields
+       and local variables.
+
+       * typemanager.cs (LookupDeclSpace): Add new method, sometimes we
+       want to pull the DeclSpace out of the builder_to_declspace instead
+       of the TypeBuilder (like in TypeContainer.FindMembers).
+
+       * class.cs (TypeContainer.FindMembers): Use LookupDeclSpace
+       instead of LookupTypeContainer.  Fixes the crash on .NET for
+       looking up interface members.
+
+       * const.cs: Create our own emit context during the Definition
+       stage, so that constants are evaluated in the proper context, when
+       a recursive definition happens.
+
+2003-05-11  Martin Baulig  <martin@ximian.com>
+
+       * statement.cs (Block.CreateSwitchBlock): New method.  Creates a
+       new block for a switch section.
+       (Block.AddLabel, Block.LookupLabel): If we're a switch section, do
+       the adding/lookup in the switch block.  Fixes #39828.
+
+2003-05-09  Miguel de Icaza  <miguel@ximian.com>
+
+       * expression.cs (UnaryMutator.LoadOneAndEmitOp): Missing
+       functionality: I needed to convert the data after I had performed
+       the add/sub operation into the operands type size.
+
+       * ecore.cs (ImplicitReferenceConversion): When boxing an interface
+       pass the type for the box operation, otherwise the resulting
+       object would have been of type object.
+
+       (BoxedCast): Add constructor to specify the type to box as.
+
+2003-05-07  Miguel de Icaza  <miguel@ximian.com>
+
+       * iterators.cs: I was reusing the `count' variable inadvertently,
+       take steps to not allow this to happen.
+
+2003-05-06  Miguel de Icaza  <miguel@ximian.com>
+
+       * attribute.cs (Attribute.Resolve): Params attributes are encoded
+       by creating an array at the point where the params starts and
+       putting all those arguments there, then adjusting the size of the
+       array.
+
+2003-05-05  Miguel de Icaza  <miguel@ximian.com>
+
+       * expression.cs (New.AddressOf): Implement interface
+       IMemoryLocation.  This is used when the `new' operator is used in
+       the context of an invocation to a method on a value type.
+
+       See http://bugzilla.ximian.com/show_bug.cgi?id=#42390 for an
+       example. 
+
+       * namespace.cs: Also check the using aliases here.
+
+       * driver.cs: Move the test for using validity after the types have
+       been entered, so we do a single pass that also includes the using
+       aliases. 
+
+       * statement.cs (Try.Resolve): Avoid crashing if there is a failure
+       in the regular case.   CreateSiblingForFinally is doing extra
+       error checking.
+
+       * attribute.cs (GetAttributeArgumentExpression): Store the result
+       on an out value, and use the return value to indicate failure
+       instead of using null (which is a valid return for Constant.GetValue).
+
+       * statement.cs: Perform the analysis flow for the increment
+       portion after the statement, because this will be the real flow of
+       execution.  Fixes #42385
+
+       * codegen.cs (EmitContext.EmitArgument,
+       EmitContext.EmitStoreArgument): New helper functions when the
+       RemapToProxy flag is set.
+
+       * expression.cs (ParameterReference.EmitLdarg): Expose this useful
+       function.
+
+       Add support for remapping parameters. 
+
+       * iterators.cs: Propagate parameter values;  Store parameter
+       values in the proxy classes.
+       
+2003-05-04  Miguel de Icaza  <miguel@ximian.com>
+
+       * ecore.cs (FieldExpr): Fix an obvious bug.  static fields do not
+       need a proxy reference;  I do not know what I was thinking
+
+       * cs-parser.jay (constructor_initializer): catch another error,
+       and display nice message.
+       
+       (field_declaration): catch void field declaration
+       to flag a better error. 
+
+       * class.cs (MemberBase.CheckBase): Report an error instead of a
+       warning if a new protected member is declared in a struct. 
+       (Field.Define): catch the error of readonly/volatile.
+
+       * ecore.cs (FieldExpr.EmitAssign): reuse the field lookup.
+
+       (FieldExpr.AddressOf): ditto.  Catch error where the address of a
+       volatile variable is taken
+
+2003-05-02  Miguel de Icaza  <miguel@ximian.com>
+
+       * statement.cs (Fixed.Resolve): Report an error if we are not in
+       an unsafe context.
+
+2003-05-01  Miguel de Icaza  <miguel@ximian.com>
+
+       * typemanager.cs: reuse the code that handles type clashes for
+       delegates and enumerations.
+
+       * class.cs (Report28): Always report.
+
+       * expression.cs (EncodeAsAttribute): Allow nulls here.
+
+2003-04-28  Miguel de Icaza  <miguel@ximian.com>
+
+       * attribute.cs (Attribute.GetAttributeArgumentExpression): Moved
+       the functionality for testing whether an expression is valid for
+       an attribute here.  Also handle the case of arrays of elements
+       being stored. 
+
+       * expression.cs (ArrayCreation.EncodeAsAttribute): Add support for
+       encoding a linear array into an array of objects that are suitable
+       to be passed to an CustomAttributeBuilder.
+
+       * delegate.cs: Check unsafe types being used outside of an Unsafe context.
+
+       * ecore.cs: (FieldExpr): Handle field remapping here.
+
+       * iteratators.cs: Pass the instance variable (if the method is an
+       instance method) to the constructors, so we can access the field
+       variables on the class.
+
+       TODO: Test this with structs.  I think the THIS variable on
+       structs might have to be a pointer, and not a refenrece
+
 2003-04-27  Miguel de Icaza  <miguel@ximian.com>
 
-       * expression.cs (ArrayCreation.CheckIndices): warn if the user has
+       * codegen.cs (EmitContext.Mapvariable): Adds a mechanism to map
+       local variables to fields in a proxy class.
+
+       * iterators.cs (PopulateProxy): Rename our internal fields to
+       <XXX>.  
+       Create a <THIS> field if we are an instance method, so we can
+       reference our parent container variables.
+       (MapVariable): Called back from the EmitContext code to enter a
+       new variable to field mapping into the proxy class (we just create
+       a FieldBuilder).
+
+       * expression.cs
+       (LocalVariableReference.{Emit,EmitAssign,AddressOf}): Add support
+       for using the remapped locals to fields.
+
+       I placed the code here, because that gives the same semantics to
+       local variables, and only changes the Emit code.
+
+       * statement.cs (Fixed.Resolve): it is not allowed to have fixed
+       statements inside iterators.
+       (VariableInfo): Add a FieldBuilder for the cases when we are
+       remapping local variables to fields in a proxy class
+
+       * ecore.cs (SimpleNameResolve): Avoid testing two times for
+       current_block != null.
+
+       * statement.cs (Swithc.SimpleSwitchEmit): Removed code that did
+       not cope with strings, as it has been moved to the
+       TableSwitchEmit.  Fixed bug in switch generation.
+
+       * expression.cs (New.DoResolve): Provide more context for the user
+       when reporting an error.
+
+       * ecore.cs (Expression.LoadFromPtr): Use ldind_i when loading
+       pointers. 
+
+       * expression.cs (MemberAccess.DoResolve): When we get a type back,
+       check the permissions for it.  Note than in a type-resolution
+       context the check was already present in DeclSpace.ResolveType,
+       but was missing from the MemberAccess.
+
+       (ArrayCreation.CheckIndices): warn if the user has
        more nested levels of expressions, but there are no more
        dimensions specified.  Avoids crash on bug 41906.