Print empty string for not mounted disks. (#148)

An empty string becomes the default value for format_not_mounted parameter
for any disk entry. This avoids printing erroneous values when a
filesystem is not mounted.

diff --git a/man/i3status.man b/man/i3status.man
index 32bc6e644cfd7c13d4aa6d360c05fa4a257088b5..d0ca2a0c2eb2a00eeba17a5640c844fdb58ec985 100644 (file)
--- a/man/i3status.man
+++ b/man/i3status.man
@@ -249,9 +249,7 @@ is assumed to be "percentage_avail" and low_threshold to be set to 0, which
 implies no coloring at all.
 
 You can define a different format with the option "format_not_mounted"
-which is used if the path does not exist or is not a mount point. So you can just empty
-the output for the given path with adding +format_not_mounted=""+
-to the config section.
+which is used if the path does not exist or is not a mount point. Defaults to "".
 
 *Example order*: +disk /mnt/usbstick+
 

diff --git a/src/print_disk_info.c b/src/print_disk_info.c
index edc69ace299639863c26515aee67d30f911abd29..624a8e2f2c2431d3074c36c0957909a8ca9dcc07 100644 (file)
--- a/src/print_disk_info.c
+++ b/src/print_disk_info.c
@@ -130,13 +130,15 @@ void print_disk_info(yajl_gen json_gen, char *buffer, const char *path, const ch
 #else
     struct statvfs buf;
 
+    if (format_not_mounted == NULL) {
+        format_not_mounted = "";
+    }
+
     if (statvfs(path, &buf) == -1) {
         /* If statvfs errors, e.g., due to the path not existing,
          * we use the format for a not mounted device. */
-        if (format_not_mounted != NULL) {
-            format = format_not_mounted;
-        }
-    } else if (format_not_mounted != NULL) {
+        format = format_not_mounted;
+    } else {
         FILE *mntentfile = setmntent("/etc/mtab", "r");
         struct mntent *m;
         bool found = false;