From 08e2ffc2b52b36fcd18434116d717bbd4de96007 Mon Sep 17 00:00:00 2001
From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 11 May 2010 06:13:55 +0000
Subject: [PATCH] Clarified why hashCode/equals must be implemented in a
 certain way

svn changeset:13136/svn branch:6.4
---
 src/com/vaadin/data/util/BeanItemContainer.java | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/com/vaadin/data/util/BeanItemContainer.java b/src/com/vaadin/data/util/BeanItemContainer.java
index b39929ff21..6d62e892ea 100644
--- a/src/com/vaadin/data/util/BeanItemContainer.java
+++ b/src/com/vaadin/data/util/BeanItemContainer.java
@@ -37,11 +37,13 @@ import com.vaadin.data.Property.ValueChangeNotifier;
  * </p>
  * 
  * <p>
- * BeanItemContainer uses the beans themselves as identifiers. For this reason,
- * they should implement {@link Object#equals(Object)} and
- * {@link Object#hashCode()}. The {@link Object#equals(Object)} implementation
- * should be stable and not depend on the contents of the bean as this may cause
- * problems when the bean is updated.
+ * BeanItemContainer uses the beans themselves as identifiers. The
+ * {@link Object#hashCode()} of a bean is used when storing and looking up beans
+ * so it must not change during the lifetime of the bean. Typically this
+ * restricts the implementation of {@link Object#equals(Object)} as well so it
+ * does not depend on the contents of the bean. This is not strictly needed but
+ * the contract between {@code equals()} and {@code hashCode()} must be
+ * fulfilled.
  * </p>
  * 
  * <p>
-- 
2.39.5