battery: Fix remaining on Linux

The current version displays the remaining time as 'n/a'.

Fixes this regression introduced in 69b2487650782f135db76078c4a7fb841cb936ac
where current_now and power_now were incorrectly replaced with current and
power when they were moved to a macro.

New component: cat

Generically reads an arbitrary file natively.

Saves a few layers of execution in comparison to using `run_command`
with an argument like `cat ./file`.

README: Add note about FreeBSD sndio dep

radical re-formatting 3/3: Error checks

Check for `< 0` instead of `== -1`.

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

radical re-formatting 2/3: Fix blocks

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

radical re-formatting 1/3: Fix spacing

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

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/

ram: Refactor on OpenBSD

Fixes up overly complicated lines, by splitting up logic

ram: Refactor Linux perc/used

Fixes up overly complicated line, by splitting up logic

battery: Refactor remaining on OpenBSD

Fixes up overly complicated line, by splitting up logic

temp: Put sysctl into define to avoid line wraps

Long, wrapped, multi-line if statements suck to read.
This fixes readability.

various: Put paths into defines to avoid line wraps

Long, wrapped, multi-line if statements suck to read.
This fixes readability in the worst places by packing format strings for
paths into defines.

separator: kill that useless thing

Just use the format strings dude. :)

do not rely on obsolete feature

function prototype with unspecified argument is obsolete since c99.

additionally some of these function which don't take any argument were
being called with a `const char *` arg, which is UB.

fix both these issues by declararing ALL the components to accept a
`const char *`, and name the arg "unused" if it's meant to be ignored.

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.

Add 'Not charging' status support

Exists on various ThinkPads. Seems to be both a synonym for "full".

Tested on:
- ThinkPad T500 (Spenser Truex <truex@equwal.com)
- ThinkPad T420 (drkhsh <me@drkhsh.at)

Add comment about mixer oss module to README

Update LICENSE

Reset sndiod initialization flag on disconnects

If the connection is lost to the sndiod(8) daemon reset the
initialization to allow for a graceful restart.

Make volume component work on FreeBSD

- Edit compile instructions
- Reuse OpenBSD sndio implementation

Fix RAM component on FreeBSD

The current implementation uses the wrong type for
given sysctl calls leading to overflow and incorrectly
reported RAM usage. The fix is to use 'unsigned int'
which is what FreeBSD expects.

Fix keyboard_indicators() prototype

according to both the function definition and the comment in
config.def.h, keyboard_indicators() should take a format string.

Update README

Update LICENSE

Add comment for FreeBSD to config.mk

Apparently `-lkvm` is needed for swap on FreeBSD

Thanks to Micheal Buch and Jason Smith
(See thread https://lists.suckless.org/dev/1907/33594.html)

Use the sioctl_open(3) OpenBSD API to access vol

Starting with OpenBSD 6.7 regular users cannot access raw audio devices
anymore, for improved security.

Instead use the sioctl_open(3) API to access and manipulate audio
controls exposed by sndiod(8). On the first call a permanent connection
is established with the running sndiod daemon, and call-back functions
are registered which are triggered when audio controls are changed
(e.g., a USB headset is attached) or when the volume is modified. On
subsequent calls we poll for changes; if there are no volume changes
this costs virtually nothing.

Joint work with Alexandre Ratchov

Add a -1 option flag

Allow slstatus to be used by programs that can grab status by calling
an external program on a periodic basis (e.g. tmux)

Handle SIGUSR1 for forced refreshes

At some point one might want to force a refresh for example after
checking email or changing the volume. Sending a SIGUSR1 achieves this
now

Fix temperature reporting on OpenBSD

On OpenBSD although the formula is correct due to integer division a
temperature of for example 54 celsius appears as 5. this patch first
treats it as a floating point op before retaining the non decimal digits

Full battery indicator

When you reach full charge the symbol would change
to a question mark "?" as "Full" was not defined,
now it changes to an "o" instead.

Add separator module

Follow International System of Units spacing rules

Add OS-support notice to README

Add ram and swap components on FreeBSD

cpu_perc: Check for division by zero

Fix CPU frequency on OpenBSD

close file descriptors and fix return values

Add native OpenBSD support for mute/volume

Based on functionality in dstat by Joerg Jung.

wifi component on FreeBSD

Add FreeBSD support for netspeeds, entropy and ip components

Update LICENSE

Add support for cpu and uptime components on FreeBSD

Add FreeBSD support for temperature and battery components

ram: Consistency with line breaks

Fix whitespace

Consistency with 80 char limit and breaks

wifi_perc: Simplify on Linux

Remove unnecessary variable

Simplify format specifiers for uintmax_t

cpu_freq: Simplify with E-notation

Thanks to the FRIGN

cpu_freq: Change to 64 bit integers

Remove unnecessary cast to size_t

Change uint64_t to uintmax_t

battery: 64 bit ints on Linux

cpu: Use uint64_t on OpenBSD

battery: Remove unnecessary defines for paths

uptime: Improve typing and sort headers

temperature: Improve types

ram: Check for theoretical division by zero

ram: Use POSIX types

entropy: Use POSIX types for format specifiers

entropy: Improve types

Consistent paramter naming for components

slstatus.h: Fix coding style

Function declarations should include variable names.
For functions with different parameters on different platfroms they are
left out.

fmt_human: Improve types

size_t may not be enough, use 64 bit integers

Add Ian Remmler to LICENSE

Add flexible formatting to keyboard_indicators.

Updated for style.

uptime: unification of the uptime function for linux/openbsd

wifi: fixed disconnected wifi status on openbsd

Clean up header includes

 - Remove <errno.h> because related functions are in util.c now
 - Remove corresponding <string.h> if not used otherwise

Update LICENSE for Quentin

Flush the output stream instead of arbitrarily disabling buffering

This is (arguably) more elegant and it matches Xlib output logic.
Also use puts instead of printf as we don't do formating there.

Don't explicitly list each component-target

I made a mistake while reading the Posix spec. In fact, the implicit
prerequisite imposed by the .c.o-rule is evaluated as expected, even if
the .c is not given in the target rule.

This partially reverts 0efd64ffaa04715eff9c834c437562952c4531cd.

swap: removed unused include

Refactor swap.c to use getline() instead of buf-filling

This drops a lot of code and even though we are using a common function,
the specification of only the needed things keeps it very efficient and
even more efficient in memory and time than the previous solution.

Update LICENSE for Tobias

Fix object order and stop using a GNU make idiom

Explicitly list component-objects in the Makefile

There was a long tinkering process at farbfeld about this, but the sad
truth is that it's the only way to make the Makefile truly portable.
Listing it just as

   $(COM:=.o): config.mk $(REQ:=.h)

omits the dependency on the c-file itself, which incurs that strictly
speaking the object file is not depending on the source file, which is
nonsense.

You don't see strictly Posix compliant Makefiles around very often and
most use nasty GNU-extensions everywhere. It is a good idea to go ahead
as a fitting example and show how to write them portably.

Use E-notation for large powers of 10

Revert "Add basic backlight percentage support"

On OpenBSD the backlight percentage cannot be retrieved in a simple way.
The only two solutions we are aware of for now are:

 - reading from /dev/ttyC0: which isn't possible without changing
 permissions or running slstatus as root
 - linking against xcb-xrandr: which is bloat and does not work in every
 case appearently

This reverts commit 37724ac2c3f496f4736223d0d8e5d8fecb933590 for now.

ram: fixed int overflow on pagetok macro

Refactor keymap.c

 - Get rid of camel-casing
 - Don't use all-caps for variable names
 - use LEN()-macro
 - use strncmp() rather than strstr() for prefix-checking
 - clean up the tokenizer-loop and don't use copies
 - make the loop more readable by separating different breaking
   conditions
 - stricter error-checking and cleanup
 - store the layout directly with bprintf rather than having
   a separate buffer

Refactor snprintf-usage in util.c

keymap: Add braces for for loop

Add keymap to README

Add Michael Buch to LICENSE

Add keymap component

Adding a new keymap component that will
indicate the current keyboard layout (language)
and variant if any was set. I use the
standard X11 XKB APIs to retrieve and parse
the xkb_symbols set with setxkbmap.

Remove o-flag as it is redundant

Even if we extend the flag to be an n-flag with a numerical input, we
already have other Unix-tools to take care of us much more flexibly,
e.g. with head(1) you can easily get the first 10 outputs with

slstatus -s | head -n 10,

but also e.g. discard the first one and then get the consecutive 10
outputs.

For the X11-root-window-name, you can limit the runtime with timeout(1)
or a comparable solution.

This reverts commit fa8b939990ed0648f99b0cbeba4efed807160db3.

Check printf() for output errors

This way cases like

$ slstatus -s > /dev/full

are properly caught and the program terminated with the proper return
code.

netspeeds: Fix indentation

Add basic backlight percentage support

At the moment linux only, but will add support for OpenBSD as well.

Change to fmt_human in netspeeds on OpenBSD

Thanks to Hiltjo

swap.c: Refactor getstats() and its usage

Add a return value so we are able to check any errors and actually fail
when there has been a critical error instead of carrying on uselessly.

Refactor fmt_human() and fix a bug

It is not necessary to copy memory or anything. Just keep a pointer to
the active prefix-array and assign the length of the arrays to a
variable.

Make the code more readable by using a switch, be more strict when an
invalid base is passed to it and fix a small oversight in the bottom of
the code where the base 1024 was forgotten to generalized.

Merge fmt_human_2 and fmt_human_10 to one function

Now only one function, fmt_human, takes an additional argument "base".

Remove fmt from load_avgs for consistency

In a later commit something like that will be enabled for every
component

Add option to output only once and exit

battery_remaining: check for division by zero

battery_remaining: Change float to double

battery: Minor coding style fixes

- Line length
- Spacing and indentation
- No explicit checks for NULL

Increase readability for load_uvmexp return

Remove units from numbers

This is a first step to decouple formatting from information because of
two reasons:

 1. The components should only gather and return the values by design
 2. Fine grained user control should be a focus

Scaling will be implemented in a different way in a later commit.