slstatus/components/run_command.c, branch master

radical re-formatting 2/3: Fix blocks

2022-10-27T23:03:38Z

Fixes coding style. Formatting commits suck, incoherent coding style sucks more. https://suckless.org/coding_style/

radical re-formatting 0/3: Alphabetic headers

2022-10-27T23:03:20Z

Except stated otherwise in comment. Fixes coding style. Formatting commits suck, incoherent coding style sucks more. https://suckless.org/coding_style/

components/*.c: include slstatus.h

2022-10-26T21:32:07Z

this gives the compiler a chance to check weather the prototype and definiton matches or not, which would catch issues like 3c47701.

Clean up header includes

2018-05-30T17:35:37Z

- Remove because related functions are in util.c now - Remove corresponding if not used otherwise

Check return value of pclose()

2018-05-19T23:01:26Z

Add warn() and die()

2018-05-18T09:13:05Z

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.

Fix coding style

2018-05-06T20:28:56Z

- 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

run_command: Clean up

2018-05-02T06:29:36Z

Format error messages properly

2018-03-28T17:46:27Z

Make use of strerror(errno) and format all errors equally: function ['parameters']: error message

Get rid of err.h as it is not portable

2018-03-28T16:26:56Z

Replace warn() and warnx() with fprintf() and add where necessary.