commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stephen Colebourne" <scolebou...@btopenworld.com>
Subject Re: [lang] MethodUtils - assignableFrom
Date Mon, 18 Nov 2002 23:05:35 GMT
Ahh, now I remember my thinking...

ClassUtils version is non-reflection specific, thus it knows nothing about
Boolean to boolean mappings for reflection purposes.

ReflectionUtils.isCompatable() was (I think) intended to be the new
MethodUtils.isAssignmentCompatible(). However on a quick review of the code
I think I missed the vital Boolean to boolean part.

The basic aim was to say that isAssignmentCompatible() is too general to be
just in MethodUtils, but is reflection specific, so should be in
ReflectionUtils.

Stephen

----- Original Message -----
From: "robert burrell donkin" <robertburrelldonkin@blueyonder.co.uk>
To: "Jakarta Commons Developers List" <commons-dev@jakarta.apache.org>
Sent: Monday, November 18, 2002 8:12 PM
Subject: Re: [lang] MethodUtils - assignableFrom


> hi Stephen
>
> i've been taking a look at the assignable logic in ClassUtils. it's
> unfortunate that it can't be used as a replacement for
> isAssignmentCompatible in MethodUtils. the problem is that reflection
> allows the matching of primitives to their class counterparts. so, for
> example, a Boolean class parameter can be used to invoke a boolean
> parameter type.
>
> (unless you feel happy with altering the way that ClassUtils.isAssignable
> works) maybe i'll rename the method in MethodUtils, make it public and
> give it a better contract.
>
> - robert
>
> On Thursday, November 14, 2002, at 11:53 PM, Stephen Colebourne wrote:
>
> > Michael, you may want to have a look at the ClassUtils class in [lang].
> > This
> > is where I moved some of the assignableFrom logic (ie. moved out of
> > MethodUtils). ClassUtils has no tests, so if you have some for this
method
> > then that would be good ;-)
> >
> > (It was moved as its not dependent on reflection, just class
> > identification)
> >
> > Stephen
> >
> >
> > --
> > To unsubscribe, e-mail:
<mailto:commons-dev-unsubscribe@jakarta.apache.
> > org>
> > For additional commands, e-mail:
<mailto:commons-dev-help@jakarta.apache.
> > org>
> >
>
>
> --
> To unsubscribe, e-mail:
<mailto:commons-dev-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail:
<mailto:commons-dev-help@jakarta.apache.org>
>


--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message