diff options
| author | drkhsh <me@drkhsh.at> | 2022-10-28 00:13:12 +0200 |
|---|---|---|
| committer | Rafael Marçalo <raroma09@gmail.com> | 2022-10-28 22:22:21 +0100 |
| commit | 8e4058fad2ce93a0c03e44fabd08ceb36c91baee (patch) | |
| tree | 99ce5d2e33198434e6425ed83ae60743d65b6ac9 | |
| parent | 5b210b49cbdcd2f9e34292a76a1b41fb0f70e0e1 (diff) | |
ram: Refactor on OpenBSD
Fixes up overly complicated lines, by splitting up logic
| -rw-r--r-- | components/ram.c | 40 |
1 files changed, 17 insertions, 23 deletions
diff --git a/components/ram.c b/components/ram.c index 1a187a0..518c18c 100644 --- a/components/ram.c +++ b/components/ram.c @@ -107,13 +107,12 @@ struct uvmexp uvmexp; int free_pages; - if (load_uvmexp(&uvmexp)) { - free_pages = uvmexp.npages - uvmexp.active; - return fmt_human(pagetok(free_pages, uvmexp.pageshift) * - 1024, 1024); - } + if (!load_uvmexp(&uvmexp)) + return NULL; - return NULL; + free_pages = uvmexp.npages - uvmexp.active; + return fmt_human(pagetok(free_pages, uvmexp.pageshift) * + 1024, 1024); } const char * @@ -122,12 +121,11 @@ struct uvmexp uvmexp; int percent; - if (load_uvmexp(&uvmexp)) { - percent = uvmexp.active * 100 / uvmexp.npages; - return bprintf("%d", percent); - } + if (!load_uvmexp(&uvmexp)) + return NULL; - return NULL; + percent = uvmexp.active * 100 / uvmexp.npages; + return bprintf("%d", percent); } const char * @@ -135,13 +133,11 @@ { struct uvmexp uvmexp; - if (load_uvmexp(&uvmexp)) { - return fmt_human(pagetok(uvmexp.npages, - uvmexp.pageshift) * 1024, - 1024); - } + if (!load_uvmexp(&uvmexp)) + return NULL; - return NULL; + return fmt_human(pagetok(uvmexp.npages, + uvmexp.pageshift) * 1024, 1024); } const char * @@ -149,13 +145,11 @@ { struct uvmexp uvmexp; - if (load_uvmexp(&uvmexp)) { - return fmt_human(pagetok(uvmexp.active, - uvmexp.pageshift) * 1024, - 1024); - } + if (!load_uvmexp(&uvmexp)) + return NULL; - return NULL; + return fmt_human(pagetok(uvmexp.active, + uvmexp.pageshift) * 1024, 1024); } #elif defined(__FreeBSD__) #include <sys/sysctl.h> |