From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 11 May 2010 06:26:48 +0000 (+0000)
Subject: Clarified javadoc for #4595
X-Git-Tag: 6.7.0.beta1~1673
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=5f48109094926545384c9c33a680386784ebec89;p=vaadin-framework.git

Clarified javadoc for #4595

svn changeset:13137/svn branch:6.4
---

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>