Comment 1 Karen Butzke 2012-09-26 10:55:48 EDT

Created attachment 221515 [details]
proposed patch against 4_2_maintenance

This patch adds an additional check for one Object being null and the other being an empty array. Thus it will return true for this use case.

I think the javadoc comment of Util.equals should be updated as well. It claims that the "must not be <code>null</code>". But then the javadoc for each parameter says they can be null.

Comment 2 Paul Webster 2012-10-26 09:20:50 EDT

That's an API method, and I'm loathe to change it's behaviour.  Perhaps we should fix it in BindingManagerEvent

PW

Comment 3 Paul Webster 2012-10-26 09:25:27 EDT

back to bug 390379

Comment 4 Dani Megert 2012-10-26 09:33:35 EDT

(In reply to comment #2)
> That's an API method, and I'm loathe to change it's behaviour.  Perhaps we
> should fix it in BindingManagerEvent
> 
> PW

+1.

Comment 5 Karen Butzke 2012-10-26 09:49:58 EDT

+1 your fix in BindingManagerEvent looks good to me