Merge pull request #3 from conduktor/remove_leaky_method_2

Remove memory-leaking `EventTarget.tableview` extension function && Bump version to v2.0.2-cdk

pom.xml

@@ -3,7 +3,7 @@
 
     <groupId>no.tornado</groupId>
     <artifactId>tornadofx</artifactId>
-    <version>2.0.1-cdk</version>
+    <version>2.0.2-cdk</version>
     <packaging>jar</packaging>
     <name>TornadoFX</name>
     <description>Lightweight JavaFX Framework for Kotlin</description>

src/main/java/tornadofx/ItemControls.kt

@@ -192,22 +192,6 @@ fun <T> EventTarget.tableview(items: ObservableList<T>? = null, op: TableView<T>
     }
 }
 
-fun <T> EventTarget.tableview(items: ReadOnlyListProperty<T>, op: TableView<T>.() -> Unit = {}) = tableview(items as ObservableValue<ObservableList<T>>, op)
-
-fun <T> EventTarget.tableview(items: ObservableValue<out ObservableList<T>>, op: TableView<T>.() -> Unit = {}) = TableView<T>().attachTo(this, op) {
-    fun rebinder() {
-        (it.items as? SortedFilteredList<T>)?.bindTo(it)
-    }
-    it.itemsProperty().bind(items)
-    rebinder()
-    it.itemsProperty().onChange {
-        rebinder()
-    }
-    items.onChange {
-        rebinder()
-    }
-}
-
 fun <T> EventTarget.treeview(root: TreeItem<T>? = null, op: TreeView<T>.() -> Unit = {}) = TreeView<T>().attachTo(this, op) {
     if (root != null) it.root = root
 }