library: eliminate a useless conditional in readproc.h
authorJim Warner <james.warner@comcast.net>
Thu, 18 Feb 2021 06:00:00 +0000 (00:00 -0600)
committerCraig Small <csmall@dropbear.xyz>
Tue, 23 Feb 2021 11:07:48 +0000 (22:07 +1100)
When #define QUICK_THREADS was introduced, for copying
some task data for a child thread, one proc_t pad byte
was used to mark, then later identify, those children.

Later the QUICK_THREADS was recycled as FALSE_THREADS,
and used for a different purpose, but a conditional in
the header file erroneously remained. Now, it is gone!

Reference(s):
. Jul, 2016 - QUICK_THREADS become FALSE_THREADS
commit c546d9dd4409ee11cd466c99a820a3b5dadfe3f4
. Aug, 2011 - QUICK_THREADS intruduced
commit bb4f08ba297a67a043f7547670aa8623b54c2e67

Signed-off-by: Jim Warner <james.warner@comcast.net>
proc/readproc.h

index 25be6d65e44cf2763173616992640be992878225..46cb825a3f39825d103e70c255bf7f11e2fd99cd 100644 (file)
@@ -35,11 +35,7 @@ typedef struct proc_t {
         ppid;           // stat,status     pid of parent process
     char
         state,          // stat,status     single-char code for process state (S=sleeping)
-#ifdef FALSE_THREADS
-        pad_1,          // n/a             padding (psst, also used if multi-threaded)
-#else
         pad_1,          // n/a             padding
-#endif
         pad_2,          // n/a             padding
         pad_3;          // n/a             padding
     unsigned long long