From 5b28b30ab191804df2b0ff8ad947dc81ab873c45 Mon Sep 17 00:00:00 2001 From: shadowwingz Date: Sun, 25 Jul 2021 10:21:20 +0800 Subject: [PATCH 1/2] Add default case --- .../tableview/popup/ColumnHeaderLongPressPopup.java | 2 ++ .../tableview/popup/RowHeaderLongPressPopup.java | 2 ++ 2 files changed, 4 insertions(+) diff --git a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java index e9865e97..f951afcb 100644 --- a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java +++ b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java @@ -115,6 +115,8 @@ public boolean onMenuItemClick(MenuItem menuItem) { case SCROLL_ROW: mTableView.scrollToRowPosition(5); break; + default: + break; } return true; } diff --git a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java index 609c4a1e..6e937429 100644 --- a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java +++ b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java @@ -97,6 +97,8 @@ public boolean onMenuItemClick(MenuItem menuItem) { case REMOVE_ROW: mTableView.getAdapter().removeRow(mRowPosition); break; + default: + break; } return true; } From 7e7d232b74b293eb5e7076aad3431d4bc09e018c Mon Sep 17 00:00:00 2001 From: shadowwingz Date: Wed, 18 Aug 2021 23:06:10 +0800 Subject: [PATCH 2/2] Return false in default case --- .../tableview/popup/ColumnHeaderLongPressPopup.java | 2 +- .../tableview/popup/RowHeaderLongPressPopup.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java index f951afcb..40dc04d2 100644 --- a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java +++ b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/ColumnHeaderLongPressPopup.java @@ -116,7 +116,7 @@ public boolean onMenuItemClick(MenuItem menuItem) { mTableView.scrollToRowPosition(5); break; default: - break; + return false; } return true; } diff --git a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java index 6e937429..c599b851 100644 --- a/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java +++ b/app/src/main/java/com/evrencoskun/tableviewsample/tableview/popup/RowHeaderLongPressPopup.java @@ -98,7 +98,7 @@ public boolean onMenuItemClick(MenuItem menuItem) { mTableView.getAdapter().removeRow(mRowPosition); break; default: - break; + return false; } return true; }