Code style cleanup and documentation/comments

This commit is contained in:
Benny Baumann 2020-12-01 22:38:13 +01:00 committed by BenBE
parent b4b952d78d
commit e3b6049043
1 changed files with 47 additions and 29 deletions

View File

@ -78,7 +78,9 @@ void ProcessList_setPanel(ProcessList* this, Panel* panel) {
void ProcessList_printHeader(ProcessList* this, RichString* header) { void ProcessList_printHeader(ProcessList* this, RichString* header) {
RichString_prune(header); RichString_prune(header);
const ProcessField* fields = this->settings->fields; const ProcessField* fields = this->settings->fields;
for (int i = 0; fields[i]; i++) { for (int i = 0; fields[i]; i++) {
const char* field = Process_fields[fields[i]].title; const char* field = Process_fields[fields[i]].title;
if (!field) { if (!field) {
@ -130,11 +132,11 @@ void ProcessList_remove(ProcessList* this, Process* p) {
} }
Process* ProcessList_get(ProcessList* this, int idx) { Process* ProcessList_get(ProcessList* this, int idx) {
return (Process*) (Vector_get(this->processes, idx)); return (Process*)Vector_get(this->processes, idx);
} }
int ProcessList_size(ProcessList* this) { int ProcessList_size(ProcessList* this) {
return (Vector_size(this->processes)); return Vector_size(this->processes);
} }
// ProcessList_updateTreeSetLayer sorts this->displayTreeSet, // ProcessList_updateTreeSetLayer sorts this->displayTreeSet,
@ -189,12 +191,14 @@ static void ProcessList_updateTreeSetLayer(ProcessList* this, unsigned int leftB
if (proc->tree_depth == deep && proc->tree_left > left && proc->tree_right < right) { if (proc->tree_depth == deep && proc->tree_left > left && proc->tree_right < right) {
if (Vector_size(layer) > 0) { if (Vector_size(layer) > 0) {
Process* previous_process = (Process*)Vector_get(layer, Vector_size(layer) - 1); Process* previous_process = (Process*)Vector_get(layer, Vector_size(layer) - 1);
// Make a 'right_bound' of previous_process in a layer a current's process index.
// Make a 'right_bound' of previous_process in a layer the current process's index.
// //
// Use 'tree_depth' as a temporal variable. // Use 'tree_depth' as a temporal variable.
// it is save to do as later 'tree_depth' will be renovated. // it is save to do as later 'tree_depth' will be renovated.
previous_process->tree_depth = proc->tree_index; previous_process->tree_depth = proc->tree_index;
} }
Vector_add(layer, proc); Vector_add(layer, proc);
} }
} }
@ -219,7 +223,7 @@ static void ProcessList_updateTreeSetLayer(ProcessList* this, unsigned int leftB
int level = deep == 0 ? 0 : (int)deep - 1; int level = deep == 0 ? 0 : (int)deep - 1;
int currentIndent = indent == -1 ? 0 : indent | (1 << level); int currentIndent = indent == -1 ? 0 : indent | (1 << level);
int nextIndent = indent == -1 ? 0 : (i < size - 1) ? currentIndent : indent; int nextIndent = indent == -1 ? 0 : ((i < size - 1) ? currentIndent : indent);
unsigned int newLeftBound = proc->tree_index; unsigned int newLeftBound = proc->tree_index;
unsigned int newRightBound = proc->tree_depth; unsigned int newRightBound = proc->tree_depth;
@ -273,9 +277,9 @@ static void ProcessList_buildTreeBranch(ProcessList* this, pid_t pid, int level,
Vector* children = Vector_new(Class(Process), false, DEFAULT_SIZE); Vector* children = Vector_new(Class(Process), false, DEFAULT_SIZE);
for (int i = Vector_size(this->processes) - 1; i >= 0; i--) { for (int i = Vector_size(this->processes) - 1; i >= 0; i--) {
Process* process = (Process*) (Vector_get(this->processes, i)); Process* process = (Process*)Vector_get(this->processes, i);
if (process->show && Process_isChildOf(process, pid)) { if (process->show && Process_isChildOf(process, pid)) {
process = (Process*) (Vector_take(this->processes, i)); process = (Process*)Vector_take(this->processes, i);
Vector_add(children, process); Vector_add(children, process);
} }
} }
@ -283,7 +287,7 @@ static void ProcessList_buildTreeBranch(ProcessList* this, pid_t pid, int level,
int size = Vector_size(children); int size = Vector_size(children);
for (int i = 0; i < size; i++) { for (int i = 0; i < size; i++) {
int index = (*node_index)++; int index = (*node_index)++;
Process* process = (Process*) (Vector_get(children, i)); Process* process = (Process*)Vector_get(children, i);
int lft = (*node_counter)++; int lft = (*node_counter)++;
@ -333,43 +337,52 @@ static long ProcessList_treeProcessCompareByPID(const void* v1, const void* v2)
return SPACESHIP_NUMBER(p1->pid, p2->pid); return SPACESHIP_NUMBER(p1->pid, p2->pid);
} }
// Builds a sorted tree from scratch, without relying on previously gathered information
static void ProcessList_buildTree(ProcessList* this) { static void ProcessList_buildTree(ProcessList* this) {
int node_counter = 1; int node_counter = 1;
int node_index = 0; int node_index = 0;
int direction = this->settings->direction; int direction = this->settings->direction;
// Sort by PID // Sort by PID
Vector_quickSortCustomCompare(this->processes, ProcessList_treeProcessCompareByPID); Vector_quickSortCustomCompare(this->processes, ProcessList_treeProcessCompareByPID);
int vsize = Vector_size(this->processes); int vsize = Vector_size(this->processes);
// Find all processes whose parent is not visible // Find all processes whose parent is not visible
int size; int size;
while ((size = Vector_size(this->processes))) { while ((size = Vector_size(this->processes))) {
int i; int i;
for (i = 0; i < size; i++) { for (i = 0; i < size; i++) {
Process* process = (Process*)(Vector_get(this->processes, i)); Process* process = (Process*)Vector_get(this->processes, i);
// Immediately consume not shown processes
// Immediately consume processes hidden from view
if (!process->show) { if (!process->show) {
process = (Process*)(Vector_take(this->processes, i)); process = (Process*)Vector_take(this->processes, i);
process->indent = 0; process->indent = 0;
process->tree_depth = 0; process->tree_depth = 0;
process->tree_left = (node_counter)++; process->tree_left = node_counter++;
process->tree_index = (node_index)++; process->tree_index = node_index++;
Vector_add(this->processes2, process); Vector_add(this->processes2, process);
ProcessList_buildTreeBranch(this, process->pid, 0, 0, direction, false, &node_counter, &node_index); ProcessList_buildTreeBranch(this, process->pid, 0, 0, direction, false, &node_counter, &node_index);
process->tree_right = (node_counter)++; process->tree_right = node_counter++;
Hashtable_put(this->displayTreeSet, process->tree_index, process); Hashtable_put(this->displayTreeSet, process->tree_index, process);
break; break;
} }
pid_t ppid = Process_getParentPid(process); pid_t ppid = Process_getParentPid(process);
// Bisect the process vector to find parent // Bisect the process vector to find parent
int l = 0, r = size; int l = 0;
int r = size;
// If PID corresponds with PPID (e.g. "kernel_task" (PID:0, PPID:0) // If PID corresponds with PPID (e.g. "kernel_task" (PID:0, PPID:0)
// on Mac OS X 10.11.6) cancel bisecting and regard this process as // on Mac OS X 10.11.6) cancel bisecting and regard this process as
// root. // root.
if (process->pid == ppid) if (process->pid == ppid)
r = 0; r = 0;
while (l < r) { while (l < r) {
int c = (l + r) / 2; int c = (l + r) / 2;
pid_t pid = ((Process*)(Vector_get(this->processes, c)))->pid; pid_t pid = ((Process*)Vector_get(this->processes, c))->pid;
if (ppid == pid) { if (ppid == pid) {
break; break;
} else if (ppid < pid) { } else if (ppid < pid) {
@ -378,29 +391,34 @@ static void ProcessList_buildTree(ProcessList* this) {
l = c + 1; l = c + 1;
} }
} }
// If parent not found, then construct the tree with this root
// If parent not found, then construct the tree with this node as root
if (l >= r) { if (l >= r) {
process = (Process*)(Vector_take(this->processes, i)); process = (Process*)Vector_take(this->processes, i);
process->indent = 0; process->indent = 0;
process->tree_depth = 0; process->tree_depth = 0;
process->tree_left = (node_counter)++; process->tree_left = node_counter++;
process->tree_index = (node_index)++; process->tree_index = node_index++;
Vector_add(this->processes2, process); Vector_add(this->processes2, process);
Hashtable_put(this->displayTreeSet, process->tree_index, process); Hashtable_put(this->displayTreeSet, process->tree_index, process);
ProcessList_buildTreeBranch(this, process->pid, 0, 0, direction, process->showChildren, &node_counter, &node_index); ProcessList_buildTreeBranch(this, process->pid, 0, 0, direction, process->showChildren, &node_counter, &node_index);
process->tree_right = (node_counter)++; process->tree_right = node_counter++;
break; break;
} }
} }
// There should be no loop in the process tree // There should be no loop in the process tree
assert(i < size); assert(i < size);
} }
assert(Vector_size(this->processes2) == vsize); (void)vsize;
assert(Vector_size(this->processes) == 0);
// Swap listings around // Swap listings around
Vector* t = this->processes; Vector* t = this->processes;
this->processes = this->processes2; this->processes = this->processes2;
this->processes2 = t; this->processes2 = t;
// Check consistency of the built structures
assert(Vector_size(this->processes) == vsize); (void)vsize;
assert(Vector_size(this->processes2) == 0);
} }
void ProcessList_sort(ProcessList* this) { void ProcessList_sort(ProcessList* this) {