Commit Graph
4 Commits
Author SHA1 Message Date
MaxandLukas Fleischer f6090c7e17 add missing call to va_end() for copied va_list
The man page stdarg(3) states that each invocation of `va_copy()` must
be matched by a corresponding invocation of `va_end()` in the same
function.

Signed-off-by: Max <max@mxzero.net>
Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>
2023-11-06 15:42:59 -05:00
MaxandLukas Fleischer daa7da0d0b correctness: kill returns -1 on error
Signed-off-by: Max <max@mxzero.net>
Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>
2023-11-06 15:42:59 -05:00
MaxandLukas Fleischer b9a0719117 remove unnecessary set of sidebar width
When the function is called, `col` is zero, which leads to setting the
sidebar width (in percent) to zero. As the variable holding the
percentage is `static unsigned sbarwidth_perc` it is initialized to zero
anyway and we set the percentage later to the value loaded from the
configuration, too.

Signed-off-by: Max <max@mxzero.net>
Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>
2023-11-06 15:42:59 -05:00
MaxandLukas Fleischer d4eb9f85d0 Remove new line character from result of password command
For example, using `pass caldav` as `PasswordCommand` returns the
password for the account including a newline character at the end
because the whole output is captured. This fix removes the new line
character at the end of the line while keeping other whitespace
character should the password contain any at the end.

Signed-off-by: Max <max@mxzero.net>
Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>
2023-11-06 15:42:59 -05:00