refactor(kotlin): simplify IconButtonTableCellEditor class initializa…

…tion #86
phodal · Sep 14, 2024 · 4269591 · 4269591
1 parent e868010
commit 4269591
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 17 deletions.
diff --git a/src/main/kotlin/com/phodal/shire/marketplace/IconButtonTableCellEditor.kt b/src/main/kotlin/com/phodal/shire/marketplace/IconButtonTableCellEditor.kt
@@ -23,22 +23,12 @@ import javax.swing.JTable
 import javax.swing.event.ChangeEvent
 import javax.swing.table.TableCellEditor
 
-open class IconButtonTableCellEditor protected constructor(
-    value: Any,
-    icon: Icon,
-    tooltipText: String,
-) :
-    AbstractCellEditor(), TableCellEditor {
-    protected val myButton: IconButton
-    protected var myValue: Any?
-
-    init {
-        myButton = IconButton(icon)
-
-        myButton.setOpaque(true)
-        myButton.setToolTipText(tooltipText)
-
-        myValue = value
+open class IconButtonTableCellEditor(value: Any, icon: Icon, tooltipText: String) : AbstractCellEditor(),
+    TableCellEditor {
+    protected var myValue: Any = value
+    protected val myButton: IconButton = IconButton(icon).apply {
+        setOpaque(true)
+        setToolTipText(tooltipText)
     }
 
     @get:VisibleForTesting

diff --git a/src/main/kotlin/com/phodal/shire/marketplace/IconButtonTableCellRenderer.kt b/src/main/kotlin/com/phodal/shire/marketplace/IconButtonTableCellRenderer.kt
@@ -166,7 +166,7 @@ interface IconTableCell {
     fun setIcon(icon: Icon?)
 }
 
-class IconButton internal constructor(defaultIcon: Icon) : JButton(defaultIcon), IconTableCell {
+class IconButton(defaultIcon: Icon) : JButton(defaultIcon), IconTableCell {
     private var myDefaultIcon: Icon
 
     init {