From e1f4a1215aa6cc65822e7968cfe385fe0b1b695c Mon Sep 17 00:00:00 2001
From: Artur Signell <artur@vaadin.com>
Date: Tue, 22 Nov 2011 16:47:20 +0200
Subject: [PATCH] FIXME comment for the future

---
 src/com/vaadin/ui/CheckBox.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/com/vaadin/ui/CheckBox.java b/src/com/vaadin/ui/CheckBox.java
index bb3a0cc66a..11d9619c8c 100644
--- a/src/com/vaadin/ui/CheckBox.java
+++ b/src/com/vaadin/ui/CheckBox.java
@@ -122,6 +122,9 @@ public class CheckBox extends AbstractField {
      */
     @Deprecated
     public boolean booleanValue() {
+        // FIXME: How should null really be handled? A default converter that
+        // converts it to false? The only UI values supported are true and
+        // false.
         Boolean value = (Boolean) getValue();
         return (null == value) ? false : value.booleanValue();
     }
-- 
2.39.5