diff options
author | Sven Eden <yamakuzure@gmx.net> | 2013-09-19 17:31:43 +0200 |
---|---|---|
committer | Sven Eden <yamakuzure@gmx.net> | 2013-09-19 17:31:43 +0200 |
commit | af1f923685f26a000dc191a3db9c6ec85ff8557c (patch) | |
tree | 1f8634db01ef5507c172d293eeb123cfc5412206 | |
parent | drawing a flag witrh wrapped descriptions does no longer add an offset for th... (diff) | |
download | ufed-af1f923685f26a000dc191a3db9c6ec85ff8557c.tar.gz ufed-af1f923685f26a000dc191a3db9c6ec85ff8557c.tar.bz2 ufed-af1f923685f26a000dc191a3db9c6ec85ff8557c.zip |
calculateDescWrap() is no aware of the brackets that are drawn around the package lists
-rw-r--r-- | ufed-curses-checklist.c | 18 | ||||
-rw-r--r-- | 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) ) ) { |