From: Max Kellermann <max@duempel.org>
Date: Mon, 6 Oct 2008 12:56:13 +0000 (+0200)
Subject: wreadln: return early from cursor movement functions
X-Git-Tag: v0.12_alpha1~54
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=32ac49ed95f5959d2206bf4e710350d435a04d05;hp=eb28e1a9398a5423e247eeda12594fa0f0a0136b;p=ncmpc.git

wreadln: return early from cursor movement functions

Unclutter these functions by removing one indent level.
---

diff --git a/src/wreadln.c b/src/wreadln.c
index 6c66895..59026ed 100644
--- a/src/wreadln.c
+++ b/src/wreadln.c
@@ -74,22 +74,24 @@ wrln_gcmp_post_cb_t wrln_post_completion_callback = NULL;
 /* move the cursor one step to the right */
 static inline void cursor_move_right(struct wreadln *wr)
 {
-	if (wr->cursor < strlen(wr->line)) {
-		++wr->cursor;
-		if (wr->cursor >= (size_t)wr->width &&
-		    wr->start < wr->cursor - wr->width + 1)
-			++wr->start;
-	}
+	if (wr->line[wr->cursor] == 0)
+		return;
+
+	++wr->cursor;
+	if (wr->cursor >= (size_t)wr->width &&
+	    wr->start < wr->cursor - wr->width + 1)
+		++wr->start;
 }
 
 /* move the cursor one step to the left */
 static inline void cursor_move_left(struct wreadln *wr)
 {
-	if (wr->cursor > 0) {
-		if (wr->cursor == wr->start && wr->start > 0)
-			--wr->start;
-		--wr->cursor;
-	}
+	if (wr->cursor == 0)
+		return;
+
+	if (wr->cursor == wr->start && wr->start > 0)
+		--wr->start;
+	--wr->cursor;
 }
 
 /* move the cursor to the end of the line */