fallingsky04
/
slstatus
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
slstatus/components/run_command.c

32 lines
509 B
C
Raw Normal View History

Added LICENSE statements to all source files
2017-09-17 15:45:03 +00:00
/* See LICENSE file for copyright and license details. */
Split into multiple files For multiple reasons the program is now split: - Make future porting to OpenBSD easier - Assign header includes to individiual functions - Make future program extensions easier - Recompile only changed parts
2017-09-17 14:18:17 +00:00
#include <stdio.h>
#include <string.h>
components/*.c: include slstatus.h this gives the compiler a chance to check weather the prototype and definiton matches or not, which would catch issues like 3c47701.
2022-10-26 20:14:53 +00:00
#include "../slstatus.h"
radical re-formatting 0/3: Alphabetic headers Except stated otherwise in comment. Fixes coding style. Formatting commits suck, incoherent coding style sucks more. https://suckless.org/coding_style/
2022-10-27 21:18:30 +00:00
#include "../util.h"
Split into multiple files For multiple reasons the program is now split: - Make future porting to OpenBSD easier - Assign header includes to individiual functions - Make future program extensions easier - Recompile only changed parts
2017-09-17 14:18:17 +00:00
const char *
run_command(const char *cmd)
{
char *p;
FILE *fp;
run_command: Clean up
2018-05-02 06:29:36 +00:00
if (!(fp = popen(cmd, "r"))) {
Add warn() and die() Given slstatus is a tool that runs in the background, most likely run from .xinitrc, it's important to prepend the name of the tool to error messages so it becomes clear where the error is coming from. To make this much more consistent, this commit adds warn() and die() utility functions consistent with other suckless projects and adapts all calls to fprintf(stderr, *) to the warn() and die() functions, greatly increasing the readability of the code.
2018-05-18 08:59:05 +00:00
warn("popen '%s':", cmd);
Split into multiple files For multiple reasons the program is now split: - Make future porting to OpenBSD easier - Assign header includes to individiual functions - Make future program extensions easier - Recompile only changed parts
2017-09-17 14:18:17 +00:00
return NULL;
}
p = fgets(buf, sizeof(buf) - 1, fp);
Check return value of pclose()
2018-05-19 22:57:24 +00:00
if (pclose(fp) < 0) {
warn("pclose '%s':", cmd);
return NULL;
}
Fix coding style - Use block for single statement ifs - Keep lines to reasonable length (current debate as to reasonable) - When functions return -1 for error test against 0 not -1 - Do not indent cases another level - Do not test against NULL and 0 explicitly - Use tabs for indentation, use spaces for alignment
2018-05-06 20:28:56 +00:00
if (!p) {
Split into multiple files For multiple reasons the program is now split: - Make future porting to OpenBSD easier - Assign header includes to individiual functions - Make future program extensions easier - Recompile only changed parts
2017-09-17 14:18:17 +00:00
return NULL;
Fix coding style - Use block for single statement ifs - Keep lines to reasonable length (current debate as to reasonable) - When functions return -1 for error test against 0 not -1 - Do not indent cases another level - Do not test against NULL and 0 explicitly - Use tabs for indentation, use spaces for alignment
2018-05-06 20:28:56 +00:00
}
if ((p = strrchr(buf, '\n'))) {
Split into multiple files For multiple reasons the program is now split: - Make future porting to OpenBSD easier - Assign header includes to individiual functions - Make future program extensions easier - Recompile only changed parts
2017-09-17 14:18:17 +00:00
p[0] = '\0';
Fix coding style - Use block for single statement ifs - Keep lines to reasonable length (current debate as to reasonable) - When functions return -1 for error test against 0 not -1 - Do not indent cases another level - Do not test against NULL and 0 explicitly - Use tabs for indentation, use spaces for alignment
2018-05-06 20:28:56 +00:00
}
Split into multiple files For multiple reasons the program is now split: - Make future porting to OpenBSD easier - Assign header includes to individiual functions - Make future program extensions easier - Recompile only changed parts
2017-09-17 14:18:17 +00:00
return buf[0] ? buf : NULL;
}