From af1f923685f26a000dc191a3db9c6ec85ff8557c Mon Sep 17 00:00:00 2001
From: Sven Eden <yamakuzure@gmx.net>
Date: Thu, 19 Sep 2013 17:31:43 +0200
Subject: calculateDescWrap() is no aware of the brackets that are drawn around
 the package lists

---
 ufed-curses-checklist.c | 18 ++++++------------
 ufed-curses-types.c     |  9 +++++++++
 2 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/ufed-curses-checklist.c b/ufed-curses-checklist.c
index b82e9f0..9dff8e2 100644
--- a/ufed-curses-checklist.c
+++ b/ufed-curses-checklist.c
@@ -220,8 +220,6 @@ static int drawflag(sFlag* flag, bool highlight)
 	desc[maxDescWidth] = 0x0;
 
 	// Description and wrapped lines state values
-	bool   hasBlankLeft  = false;                 // Set to true once the left side is blanked
-	bool   hasBlankRight = false;                 // Set to true once the right (state) side is blanked
 	bool   hasHead       = false;                 // Set to true once the left side (flag name and states) are printed
 	int    rightwidth    = lWidth - minwidth - 8; // Space on the right to print descriptions
 	size_t length        = rightwidth;            // Characters to print when not wrapping
@@ -246,17 +244,16 @@ static int drawflag(sFlag* flag, bool highlight)
 			continue;
 		}
 
-		// If the flag name and state are drawn, following lines
-		// need to start with spaces
-		if (hasHead && !hasBlankLeft) {
-			memset(buf, ' ', minwidth);
-			hasBlankLeft = true;
-		}
+		// Always start with a blanked buffer
+		memset(buf,  ' ', sizeof(char) * lWidth);
 
 		// Prepare new description or blank on wrapped parts
 		if (newDesc) {
 			special = getFlagSpecialChar(flag, idx);
 
+			// Always start with a blank description buffer
+			memset(desc, ' ', sizeof(char) * maxDescWidth);
+
 			// Wrapped and not wrapped lines are unified here
 			// to simplify the usage of different ordering and
 			// stripped descriptions versus original descriptions
@@ -272,9 +269,6 @@ static int drawflag(sFlag* flag, bool highlight)
 							  flag->desc[idx].pkg);
 			} else
 				sprintf(desc, "%s", flag->desc[idx].desc);
-		} else if (!hasBlankRight) {
-			memset(buf + minwidth, ' ', 10);
-			hasBlankRight = true;
 		}
 
 		/* --- Preparations done --- */
@@ -293,7 +287,7 @@ static int drawflag(sFlag* flag, bool highlight)
 
 		// The right side of buf can be added now:
 		leftover = rightwidth - (int)length;
-		pBuf     = buf + minwidth + (newDesc ? 8 : 10);
+		pBuf     = &buf[minwidth + (newDesc ? 8 : 10)];
 		sprintf(pBuf, "%-*.*s",
 			(int)length, (int)length,
 			strlen(desc) > pos ? &desc[pos] : " ");
diff --git a/ufed-curses-types.c b/ufed-curses-types.c
index 4a5445f..14c9313 100644
--- a/ufed-curses-types.c
+++ b/ufed-curses-types.c
@@ -595,6 +595,15 @@ static void calculateDescWrap(sDesc* desc)
 				--left;
 			}
 
+			// Correct length if this is the package list
+			// Note: in drawflag() the string is pre- and postfixed with '(' / ')'
+			if (pch == pPkg) {
+				if (!curr->pos)
+					++curr->len;
+				if (end >= (wLen - 1))
+					++curr->len;
+			}
+
 
 			// Step 4: Switch if the current string is exhausted:
 			if (left && (!wLen || (end >= (wLen - 1)) || (start >= wLen) ) ) {
-- 
cgit v1.2.3-65-gdbad