From c22fd8b708db5b244292f99a5b49b17d790baddd Mon Sep 17 00:00:00 2001
From: Twan van Laarhoven <twanvl@gmail.com>
Date: Mon, 1 Jun 2020 22:29:04 +0200
Subject: [PATCH] fix: return statement in all control paths

---
 src/gui/value/text.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/gui/value/text.cpp b/src/gui/value/text.cpp
index 5ff3f64a..8a6c4d36 100644
--- a/src/gui/value/text.cpp
+++ b/src/gui/value/text.cpp
@@ -426,13 +426,13 @@ bool TextValueEditor::onChar(wxKeyEvent& ev) {
       }
       return true;
     case WXK_UP:
-      if ( wordListDropDown(findWordList(selection_end_i)) ) break;
+      if (wordListDropDown(findWordList(selection_end_i))) return true;
       moveSelection(TYPE_INDEX, v.moveLine(selection_end_i, -1), !ev.ShiftDown(), MOVE_LEFT_OPT);
       return true;
     case WXK_DOWN:
-      if ( wordListDropDown(findWordList(selection_end_i)) ) break;
+      if (wordListDropDown(findWordList(selection_end_i))) return true;
       moveSelection(TYPE_INDEX, v.moveLine(selection_end_i, +1), !ev.ShiftDown(), MOVE_RIGHT_OPT);
-      break;
+      return true;
     case WXK_HOME:
       // move to begining of line / all (if control)
       if (ev.ControlDown()) {