Merge pull request #290 from duskCoder/changes

Fix potential issues & avoid unnecessary instructions

Prevent potential crash if glob() fails

Calling globfree(NULL) is undefined behaviour. In Linux (glibc), it
results in a segmentation fault.

It is also undefined behaviour to call globfree(&pglob) if a previous
call to glob(&pglob) returned an error.

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Fix potential memory leak on Linux

The function slurp_all_batteries(), on Linux, allocates memory
dynamically

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Fix invalid handling of glob() errors on Linux

The manual of glob(3) says that the function returns 0 on successful
completion. Any other integer value should be considered an error, not
only negative integers.

In practice, *BSD systems use negative values but Linux uses positive
integers.

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Fix use of undefined macro __OpenBSD__

Compiling on Linux with -Wundef produces the following warning:

  warning: "__OpenBSD__" is not defined, evaluates to 0 [-Wundef]

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Avoid assigning a new value to a var before using the old value

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Make sure the arguments passed to printf/die(...) match the format

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

No longer use a temporary buffer in the die() function

Before the following change

  f947d0a Breaks configfiles! Major refactoring of i3status, see below

The die(fmt, ...) function was outputting the reason to the status bar
in addition to stderr. For this reason, it was meaningful to create a
temporary string according to the format string and then passing it
around to the different functions.

Nowadays, we only display the error message to stderr so calling
fprintf(stderr, ...) is much simpler.

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Merge pull request #289 from bebehei/testsuite

Testsuite

Show differing results, if testcase fails

Run all travis tests and expose the final result as exit code

Expose the testsuite in the Makefile

Fail the tests, if i3status actually crashes

Merge pull request #287 from eplanet/emeric/man_typo

Fix man typo

Fix man typo

Merge pull request #286 from duskCoder/#283

Fix issue #283 (shown IP belonging to wrong network interface)

Fix shown IP address belonging to wrong interface

The following commit:

  6a75ea9 Show IP address when address has a label

introduced a way to show the IP address of an interface when a label is
associated to the IP.

When a label is associated to an IP, the structure returned by
getifaddrs() has the label concatenated to the interface name in the
.ifa_name field as in the following example:

    struct ifaddrs ifaddr = {
       .ifa_name = "eth0:mylabel",
    };

As a consequence, using a strict comparison between the interface name
and the .ifa_name field yields a falsy result. However, checking if the
.ifa_name starts with the interface name (e.g. eth0) does not work
either because other network interfaces can have a name which starts
with eth0.

This commit solves the issue by stripping out the optional label from
the .ifa_name field before making a strict comparison with the interface
name.

Fix #283

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Simplify the algorithm used to determine the IP address

Signed-off-by: Olivier Gayot <olivier.gayot@sigexec.com>

Merge pull request #282 from Stunkymonkey/mem-support

fix wrong memory-documentation

fix wrong memory-documentation

Merge pull request #270 from Stunkymonkey/mem-support

initial support of memory-usage for linux

Merge pull request #281 from eplanet/fix_warnings

Fix MacOS build

Fix MacOS build

release v2.12

Simplify the memory thresholds

Rename ram variables consistently

Implement support for memory-usage on Linux

Merge pull request #275 from winks/patch-2

Fix NULL value on OpenBSD when there is no acpitz0

Fix NULL value on OpenBSD when there is no acpitz0

Using %degrees on OpenBSD resulted in garbage values if sysctl doesn't
know about any cpu temperature sensors.

Merge pull request #213 from dlrobertson/addr_description

Show IP address when address has a label

Show IP address when address has a label

If an address has been added with a label, ensure the correct address is
shown.

Merge pull request #264 from foutaise/master

Make iface_type() work on FreeBSD

Merge branch 'master' into master

Merge pull request #251 from tomaskrizek/man-eth-wireless-format

man: explicitly use format_up/format_down in ethernet, wireless conf

Merge pull request #243 from jasperla/fei_error

Remove overly aggressive #error

Merge pull request #266 from moreais/master

OpenBSD: fix volume display on some hardware

OpenBSD: fix volume display on some hardware

On some hardware the AUDIO_MIXER_READ ioctl requires the current number
of channels to be set, and the mute device found needs to be checked if
it belongs to the master output device.

Make iface_type() work on FreeBSD

Merge pull request #262 from foutaise/master

Add check for interval value

Add check for interval value

Merge pull request #261 from tylerbrazier/no-sloc

Remove SLOC comment from man page

Remove SLOC comment from man page

This changed happened for the readme in
73620dc876d7f76b9a6771048c5ad0ade94e5bd7

Treat zero battery capacity as "not available" (#259)

`print_battery_info` computes `batt_info.percentage_remaining` by
dividing batt_info.remaining by `full`. If `full` is `0` then the
battery remaining will be reported as "inf".

Before this, it tries to set `full` to either the design capacity or to
the last known good charge. It determines if these values are available
by checking whether their fields in `batt_info` are non-negative. As it
initialized `batt_info` with values of `-1`, a non-negative value
implies that something has provided a value.

`slurp_all_batteries` and `add_battery_info` however initialize these
fields to zero, so if these functions are called then
`batt_info.full_design` will always be used.

This means that on systems that don't provide a value for design
capacity the percentage remaining will be reported as "inf", unless the
user has set `last_full_capacity` to `true` in their `i3status.conf`.

This patch changes `print_battery_info` to expect values for the battery
capacity to be strictly greater than zero. This seems reasonable as a
battery with a capacity of zero isn't useful.

An alternative solution would be to change `slurp_all_batteries` and
`add_battery_info` to initialize `batt_info` with `-1`, as
`print_battery_info` does. This is less appealing as `add_battery_info`
is accumulating the values, so using `-1` would introduce off-by-one
errors without additional code to avoid them.

Merge pull request #255 from eplanet/update_readme

Get rid of the SLOC comment

Get rid of the SLOC comment

Update comment as in #247 (#254)

Add IPv6 address when IPv4 isn't available (#247)

Merge pull request #253 from eplanet/always_reload_tz

Call tzset at each time print

Call tzset at each time print

Merge pull request #252 from GuyOfThePery/patch-1

Fixed minor typos

Fixed minor typos

"you can chose" -> "you can choose"

"You can either disable the default separator altogether setting it to the empty string." -> " You can also disable the default separator altogether by setting it to the empty string."

man: explicitly use format_up/format_down in ethernet, wireless conf

Clearly indicate format_up and format_down are used for configuring ethernet
and wireless sections instead of format.

Signed-off-by: Tomas Krizek <tomas.krizek@mailbox.org>

Add network test cases (#246)

Merge pull request #237 from kousu/215-battery

Be more cautious about handling invalid battery measurements.

Fix CPU unit tests (#239)

Support any amount of available cores on testing machine.

Makefile: make PKG_CONFIG overridable (patch by Helmut Grohne)

Remove overly aggressive #error

Change-Id: I8c20606d386d713cac51b620d1fcdb479a3622fe

Fix CPU unit tests (#239)

Support any amount of available cores on testing machine.

Repair test case.

This was introduced in #236. It looks like Travis never ran on that PR?
Anyway it's blocking me submitting this now so I'm fixing it.

Be more cautious about handling invalid battery measurements.

Use local timezone for battery %emptytime (#236)

Merge pull request #219 from flammi/master

Add check for virtual ethernet devices

Add battery capacity parsing (#216)

The Linux power supply class defines three entries to provide battery status.
One of them wasn't used: POWER_SUPPLY_CAPACITY.

https://www.kernel.org/doc/Documentation/power/power_supply_class.txt

Added braces

Added braces to the if-statement as requested.

Fix memory leak/Use BEGINS_WITH macro

The orignal proposed code had a memory leak when returning true.
Furthermore I included the handy BEGINS_WITH macro of i3 which makes the
code (IMHO) a lot more readable.

Add check for virtual ethernet devices

The _first_ option for ethernet devices now uses the link in sysfs to determine
if it's a real device or just a virtual one (i.e veth** devices created by docker).

Multiple CPU support for cpu_usage (#209)

This change addresses the issue #199 asking for multiple CPU support. It
takes an arbitrary CPU number and outputs its usage using the same
arithmetics as for CPU aggregation. It currently doesn't support
FreeBSD.

Merge pull request #211 from ixjlyons/issue210

disk: don't remove trailing slash if path is a single character

disk: don't remove trailing slash if path is a single character

Merge pull request #208 from stapelberg/warning

remove useless checks (we use char, not unsigned char)

Merge pull request #207 from stapelberg/disk_info

disk: sanitize trailing slashes

remove useless checks (we use char, not unsigned char)

fixes compilation warnings

disk: sanitize trailing slashes

fixes #200

Merge pull request #202 from jasperla/openbsd_fdleak

Plug an fd leak in the OpenBSD-specific code for _first_

Merge pull request #201 from jasperla/openbsd_bat_info

Suppress printing :00 seconds of remaining battery lifetime

Plug an fd leak in the OpenBSD-specific code for _first_

Suppress printing :00 seconds of remaining battery lifetime, as apm(4)'s
estimate only has a granularity of minutes.

From tb@openbsd.org

Fix the deciKelvin to Celsius conversion on FreeBSD (#195)

Change from ceiled value to floored value to be consistent with the
FreeBSD kernel

https://svnweb.freebsd.org/base?view=revision&revision=300421

Make first_eth_interface() work on OpenBSD: (#197)

- use a define for loopback interface device
- use an approach similar to ifconfig(8) for determining device type

Include sys/select.h on OpenBSD (#198)

Fixes a compilation error:
/usr/include/net80211/ieee80211_ioctl.h:339: warning: implicit declaration of function 'howmany'
/usr/include/net80211/ieee80211_ioctl.h:339: error: 'NBBY' undeclared here (not in a function)
/usr/include/net80211/ieee80211_ioctl.h:339: error: variably modified 'nr_rxmcs' at file scope

release v2.11

correctly write I3STATUS_VERSION on release

Makefile: sort objects for deterministic linking order

Patch by Reiner Herrmann

man: fix spelling errors

man: switch link to directly go to github

Consolidate measure speed script (#188)

* remove old net speed scripts

* add reworked net-speed script

* add space between rate and speed suffix

it is easier to parse visually

* only use intervals greater zero

* add ifaces examples

* catch negativ byte count

happens when the counter resets

Re-license contrib/netspeed to the same as i3status (#189)

Signed-off-by: Zhong Jianxin <azuwis@gmail.com>

Fix i3status to compile with -fno-common (#185)

This avoids multiple declarations of the same global variable in
different source files.

Adding documentation for PR #179 (#182)

Merge pull request #179 from mihaicmn/feature-threshold-format

Provide format_above_threshold/format_below_threshold options

Merge pull request #178 from eplanet/ut_cpu_temp

Unit testing some new cases

Add unit tests for threshold format

Add 'format_below_threshold' option for 'disk' module
Add 'format_above_threshold' option for 'cpu_temperature' module
Add 'format_above_threshold' option for 'cpu_usage' module
Add 'format_above_threshold' option for 'load' module

Added test case for run_watch

The valid test case assumes pid 1 exists, which should always
be true on Unix environment.

Added unit test case for path_exists

Added unit test case for CPU temperature

Unit testing in Travis (#170)

Use blank padding for wireless quality (#166) (#167)

Setting of custom locale in tztime configuration. (#168)

To be able to show my birth country's time zone in that country's locale, and my local time in my current locale.

Detect batteries using glob (#163)