[m-rev.] for review: comment out castclass System.Type[]
Fergus Henderson
fjh at cs.mu.OZ.AU
Sun Dec 16 17:32:35 AEDT 2001
On 14-Dec-2001, Peter Ross <peter.ross at miscrit.be> wrote:
> Peter wrote:
> > ===================================================================
> >
> >
> > Estimated hours taken: 8
> > Branches: main
> >
> > System.Reflection.Assembly.GetExportedTypes returns an array of
> > System.Type however in Beta 2 of the CLR, when you try and do a
> > castclass to System.Type[] it fails. The fix is not to do the
> > castclass.
> >
> > compiler/ilasm.m:
> > Disable castclasses where the type is System.Type[], by commenting
> > the line out.
> >
>
> I have submitted this change, because on further investigation it appears to
> be a bug.
What do you mean by "submitted"?
Do you mean "committed"?
And when you say "it appears to be a bug", do you mean that it appears
to be a bug in the CLR, or a bug in the Mercury compiler?
I think it is a bug in the Mercury compiler,
because `castclass' should not be used with array types,
but I think your change does not properly fix the bug.
--
Fergus Henderson <fjh at cs.mu.oz.au> | "I have always known that the pursuit
The University of Melbourne | of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh> | -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post: mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------
More information about the reviews
mailing list