In the ProxyHandler class, method invoke(), i see:

        if (proxyManager.isProxy(targetObject)) {
          args[0] = proxyManager.unwrap(proxy).call();

Shouldn't that be:

        if (proxyManager.isProxy(targetObject)) {
          args[0] = proxyManager.unwrap(targetObject).call();

because I think right now, all tests return true because we're
comparing the object to itself in all cases.

On Wed, Dec 22, 2010 at 14:57, Guillaume Nodet <[email protected]> wrote:
> It actually seems to work in some cases.
> Or was that fixed recently maybe ?
> I need to dig a bit more too, as I do had some broken code a few days ago.
>
> On Wed, Dec 22, 2010 at 14:26, Alasdair Nottingham <[email protected]> wrote:
>> Looking at the code the equals method should work, but there could be
>> a bug. I'll try to write a test to check this out.
>>
>> Alasdair
>>
>> On 22 December 2010 13:02, Guillaume Nodet <[email protected]> wrote:
>>> Isn't there a regression with proxies ?  I can't put blueprint proxies
>>> in a list and remove them anymore as the equals() method does not work
>>> well on those proxies.
>>>
>>> --
>>> Cheers,
>>> Guillaume Nodet
>>> ------------------------
>>> Blog: http://gnodet.blogspot.com/
>>> ------------------------
>>> Open Source SOA
>>> http://fusesource.com
>>>
>>
>>
>>
>> --
>> Alasdair Nottingham
>> [email protected]
>>
>
>
>
> --
> Cheers,
> Guillaume Nodet
> ------------------------
> Blog: http://gnodet.blogspot.com/
> ------------------------
> Open Source SOA
> http://fusesource.com
>



-- 
Cheers,
Guillaume Nodet
------------------------
Blog: http://gnodet.blogspot.com/
------------------------
Open Source SOA
http://fusesource.com

Reply via email to