From 5f48109094926545384c9c33a680386784ebec89 Mon Sep 17 00:00:00 2001
From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 11 May 2010 06:26:48 +0000
Subject: [PATCH] Clarified javadoc for #4595

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

diff --git a/src/com/vaadin/data/util/BeanItemContainer.java b/src/com/vaadin/data/util/BeanItemContainer.java
index 6d62e892ea..f80a256ff4 100644
--- a/src/com/vaadin/data/util/BeanItemContainer.java
+++ b/src/com/vaadin/data/util/BeanItemContainer.java
@@ -39,11 +39,10 @@ import com.vaadin.data.Property.ValueChangeNotifier;
  * <p>
  * 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.
+ * so it must not change during the lifetime of the bean (it should not depend
+ * on any part of the bean that can be modified). Typically this restricts the
+ * implementation of {@link Object#equals(Object)} as well in order for it to
+ * fulfill the contract between {@code equals()} and {@code hashCode()}.
  * </p>
  * 
  * <p>
-- 
2.39.5