[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