From c18309466d0974fd839c5b8c61edc590447092d4 Mon Sep 17 00:00:00 2001 From: Hisham Muhammad Date: Thu, 18 Feb 2016 14:32:49 -0200 Subject: [PATCH] Pre-reset 'show' for process. This simplifies the protocol between the platform-independent and platform-specific parts. The platform-specific parts were supposed to re-determine the value of process->show on each iteration, and the Darwin subsystem wasn't doing that. Instead of adding the code to the Darwin part, I lifted the burden of the OS-specific of resetting process->show: now they can choose to hide a process if they want to (e.g. detecting kernel threads) but are not required to (e.g. on Darwin where we're not listing threads separately (yet?)). Fixes tree view collapsing/expanding on OSX. Closes #416. --- ProcessList.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ProcessList.c b/ProcessList.c index 6218a6cd..83e200eb 100644 --- a/ProcessList.c +++ b/ProcessList.c @@ -311,6 +311,7 @@ void ProcessList_scan(ProcessList* this) { for (int i = 0; i < Vector_size(this->processes); i++) { Process* p = (Process*) Vector_get(this->processes, i); p->updated = false; + p->show = true; } this->totalTasks = 0;