commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From robert burrell donkin <robertburrelldon...@blueyonder.co.uk>
Subject Re: [lang] MethodUtils - assignableFrom
Date Tue, 19 Nov 2002 17:57:23 GMT
makes sense to me :)

i'll hunt the right method in ReflectionUtils and fix any bugs it contains.

- robert

On Monday, November 18, 2002, at 11:05 PM, Stephen Colebourne wrote:

> 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>
>


--
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