philiptaron/lix
0
0
Fork 0
forked from lix-project/lix
Code Pull requests Activity

mildly refactor the renderActivity if hell-chain

Browse source 
This is primarily for readability, but iwrc chaining std::string's
operator+ is also pretty scuffed performance-wise, and this was doing a
lot of operator+ chainging.

Change-Id: I9f25235df153eb2bbb491f1ff7ebbeed9a8ec985
This commit is contained in:
Qyriad 2024-06-20 13:50:31 -06:00
parent 5f0062285c
commit ce4dee0fa5
1 changed files with 47 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

66
src/libmain/progress-bar.cc
View file

@ -402,31 +402,59 @@ std::string ProgressBar::getStatus(State & state)
expected = std::max(expected, act.expected); expected = std::max(expected, act.expected);
std::string s; std::string rendered;
if (running || done || expected || failed) { if (running || done || expected || failed) {
if (running) if (running) {
if (expected != 0) if (expected != 0) {
s = fmt(ANSI_BLUE + numberFmt + ANSI_NORMAL "/" ANSI_GREEN + numberFmt + ANSI_NORMAL "/" + numberFmt, auto const runningPart = fmt(numberFmt, running / unit);
running / unit, done / unit, expected / unit); auto const donePart = fmt(numberFmt, done / unit);
else auto const expectedPart = fmt(numberFmt, expected / unit);
s = fmt(ANSI_BLUE + numberFmt + ANSI_NORMAL "/" ANSI_GREEN + numberFmt + ANSI_NORMAL, rendered = fmt(
running / unit, done / unit); ANSI_BLUE "%s" ANSI_NORMAL "/" ANSI_GREEN "%s" ANSI_NORMAL "/%s",
else if (expected != done) runningPart,
if (expected != 0) donePart,
s = fmt(ANSI_GREEN + numberFmt + ANSI_NORMAL "/" + numberFmt, expectedPart
done / unit, expected / unit); );
else } else {
s = fmt(ANSI_GREEN + numberFmt + ANSI_NORMAL, done / unit); auto const runningPart = fmt(numberFmt, running / unit);
else auto const donePart = fmt(numberFmt, done / unit);
s = fmt(done ? ANSI_GREEN + numberFmt + ANSI_NORMAL : numberFmt, done / unit); rendered = fmt(
s = fmt(itemFmt, s); ANSI_BLUE "%s" ANSI_NORMAL "/" ANSI_GREEN "%s" ANSI_NORMAL,
runningPart,
donePart
);
}
} else if (expected != done) {
if (expected != 0) {
auto const donePart = fmt(numberFmt, done / unit);
auto const expectedPart = fmt(numberFmt, expected / unit);
rendered = fmt(
ANSI_GREEN "%s" ANSI_NORMAL "/%s",
donePart,
expectedPart
);
} else {
auto const donePart = fmt(numberFmt, done / unit);
rendered = fmt(ANSI_GREEN "%s" ANSI_NORMAL, donePart);
}
} else {
auto const donePart = fmt(numberFmt, done / unit);
// We only color if `done` is non-zero.
if (done) {
rendered = concatStrings(ANSI_GREEN, donePart, ANSI_NORMAL);
} else {
rendered = donePart;
}
}
rendered = fmt(itemFmt, rendered);
if (failed) if (failed)
s += fmt(" (" ANSI_RED "%d failed" ANSI_NORMAL ")", failed / unit); rendered += fmt(" (" ANSI_RED "%d failed" ANSI_NORMAL ")", failed / unit);
} }
return s; return rendered;
}; };
auto showActivity = [&](ActivityType type, const std::string & itemFmt, const std::string & numberFmt = "%d", double unit = 1) { auto showActivity = [&](ActivityType type, const std::string & itemFmt, const std::string & numberFmt = "%d", double unit = 1) {