FIXME comment for the future

diff --git a/src/com/vaadin/ui/CheckBox.java b/src/com/vaadin/ui/CheckBox.java
index bb3a0cc66a4208c49b0341afa13412f23ce61466..11d9619c8c16faea553188b54125a8db138794c0 100644 (file)
--- 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();
     }