[PATCH 15/15] WINGs: removed unnecessary gotos in handleTextKeyPress

[Christophe]

From: Christophe CURIS <christophe.cu...@free.fr>

When the condition can be simply checked in an if, it is better to do that
than to use a flow breaking goto.

Signed-off-by: Christophe CURIS <christophe.cu...@free.fr>
---
 WINGs/wtext.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/WINGs/wtext.c b/WINGs/wtext.c
index c112fbe..734dc8b 100644
--- a/WINGs/wtext.c
+++ b/WINGs/wtext.c
@@ -2231,11 +2231,8 @@ static void handleTextKeyPress(Text * tPtr, XEvent * 
event)
        case XK_Left:
                if (!(tb = tPtr->currentTextBlock))
                        break;
-               if (tb->graphic)
-                       goto L_imaGFX;
 
-               if (tPtr->tpos == 0) {
- L_imaGFX:
+               if (tb->graphic || tPtr->tpos == 0) {
                        if (tb->prior) {
                                tPtr->currentTextBlock = tb->prior;
                                if (tPtr->currentTextBlock->graphic)
@@ -2256,10 +2253,7 @@ static void handleTextKeyPress(Text * tPtr, XEvent * 
event)
        case XK_Right:
                if (!(tb = tPtr->currentTextBlock))
                        break;
-               if (tb->graphic)
-                       goto R_imaGFX;
-               if (tPtr->tpos == tb->used) {
- R_imaGFX:
+               if (tb->graphic || tPtr->tpos == tb->used) {
                        if (tb->next) {
                                tPtr->currentTextBlock = tb->next;
                                tPtr->tpos = 0;
-- 
1.9.2


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.