diff options
author | Aaron Marcher <info@nulltime.net> | 2016-03-18 16:15:05 +0100 |
---|---|---|
committer | Aaron Marcher (drkhsh) <info@nulltime.net> | 2016-03-18 16:15:05 +0100 |
commit | 02af858a35d225f25de260accec19fa71587d7f2 (patch) | |
tree | bc05bdc7ccc0fdecacd83913bad8cd373b1dc858 | |
parent | 096737778bdd57e41dcc5f2c2bae2e175f86435b (diff) |
Easier info function names
All the info function names started with "get_", which I removed to make
it easier for the user to configure the program to its needs.
Additionally I renamed some functions (e.g. get_ram_usage) to better
names, making it easier to extend the program with ram usage / total
functions.
-rw-r--r-- | config.def.h | 18 | ||||
-rw-r--r-- | slstatus.c | 61 | ||||
-rw-r--r-- | slstatus.h | 16 |
3 files changed, 48 insertions, 47 deletions
diff --git a/config.def.h b/config.def.h index b3f01df..e84c40a 100644 --- a/config.def.h +++ b/config.def.h @@ -21,13 +21,13 @@ static unsigned int update_interval = 1; - volume (alsa volume and mute status in percent) [argument: soundcard] - wifi_signal (wifi signal in percent) [argument: wifi card interface name] */ static const struct arg args[] = { - /* function format argument */ - { get_wifi_signal, "wifi %4s | ", "wlp3s0" }, - { get_battery, "bat %4s | ", "BAT0" }, - { get_cpu_usage, "cpu %4s ", NULL }, - { get_cpu_temperature, "%3s | ", "/sys/devices/platform/coretemp.0/hwmon/hwmon2/temp1_input" }, - { get_ram_usage, "ram %3s | ", NULL }, - { get_volume, "vol %4s | ", "default" }, - { get_diskusage, "ssd %3s | ", "/" }, - { get_datetime, "%s", "%y-%m-%d %H:%M:%S" } + /* function format argument */ + { wifi_perc, "wifi %4s | ", "wlp3s0" }, + { battery_perc, "bat %4s | ", "BAT0" }, + { cpu_perc, "cpu %4s ", NULL }, + { temp, "%3s | ", "/sys/devices/platform/coretemp.0/hwmon/hwmon2/temp1_input" }, + { ram_perc, "ram %3s | ", NULL }, + { vol_perc, "vol %4s | ", "default" }, + { disk_perc, "ssd %3s | ", "/" }, + { datetime, "%s", "%y-%m-%d %H:%M:%S" }, }; @@ -45,7 +45,7 @@ smprintf(const char *fmt, ...) /* battery percentage */ char * -get_battery(const char *battery) +battery_perc(const char *battery) { int now, full, perc; char batterynowfile[64] = ""; @@ -95,32 +95,9 @@ get_battery(const char *battery) return smprintf("%d%%", perc); } -/* cpu temperature */ -char * -get_cpu_temperature(const char *file) -{ - int temperature; - FILE *fp; - - /* open temperature file */ - if (!(fp = fopen(file, "r"))) { - fprintf(stderr, "Could not open temperature file.\n"); - return smprintf("n/a"); - } - - /* extract temperature */ - fscanf(fp, "%d", &temperature); - - /* close temperature file */ - fclose(fp); - - /* return temperature in degrees */ - return smprintf("%d°C", temperature / 1000); -} - /* cpu percentage */ char * -get_cpu_usage(const char *null) +cpu_perc(const char *null) { int perc; long double a[4], b[4]; @@ -162,7 +139,7 @@ get_cpu_usage(const char *null) /* date and time */ char * -get_datetime(const char *timeformat) +datetime(const char *timeformat) { time_t tm; size_t bufsize = 64; @@ -186,7 +163,7 @@ get_datetime(const char *timeformat) /* disk usage percentage */ char * -get_diskusage(const char *mountpoint) +disk_perc(const char *mountpoint) { int perc = 0; struct statvfs fs; @@ -206,7 +183,7 @@ get_diskusage(const char *mountpoint) /* ram percentage */ char * -get_ram_usage(const char *null) +ram_perc(const char *null) { int perc; long total, free, buffers, cached; @@ -234,9 +211,33 @@ get_ram_usage(const char *null) return smprintf("%d%%", perc); } +/* temperature */ +char * +temp(const char *file) +{ + int temperature; + FILE *fp; + + /* open temperature file */ + if (!(fp = fopen(file, "r"))) { + fprintf(stderr, "Could not open temperature file.\n"); + return smprintf("n/a"); + } + + /* extract temperature */ + fscanf(fp, "%d", &temperature); + + /* close temperature file */ + fclose(fp); + + /* return temperature in degrees */ + return smprintf("%d°C", temperature / 1000); +} + + /* alsa volume percentage */ char * -get_volume(const char *soundcard) +vol_perc(const char *soundcard) { int mute = 0; long vol = 0, max = 0, min = 0; @@ -274,7 +275,7 @@ get_volume(const char *soundcard) /* wifi percentage */ char * -get_wifi_signal(const char *wificard) +wifi_perc(const char *wificard) { int bufsize = 255; int strength; @@ -14,11 +14,11 @@ struct arg { /* functions */ void setstatus(const char *); char *smprintf(const char *, ...); -char *get_battery(const char *); -char *get_cpu_temperature(const char *); -char *get_cpu_usage(const char *); -char *get_datetime(const char *); -char *get_diskusage(const char *); -char *get_ram_usage(const char *); -char *get_volume(const char *); -char *get_wifi_signal(const char *); +char *battery_perc(const char *); +char *cpu_perc(const char *); +char *datetime(const char *); +char *disk_perc(const char *); +char *ram_perc(const char *); +char *temp(const char *); +char *vol_perc(const char *); +char *wifi_perc(const char *); |