vmstat: -w switch for wider output

This is a rework of the merge request #5 that unconditionally
forced the output to cross the 80 chars border.
With this commit users can switch to the wide output mode
with the -w option.

diff --git a/vmstat.8 b/vmstat.8
index d61b1aa9fad06e4445f5f83ab88e236c8a08db4b..ef6cbe9940d7912f15e58adde588bb14405dccd2 100644 (file)
--- a/vmstat.8
+++ b/vmstat.8
@@ -74,6 +74,11 @@ or 1048576
 bytes.  Note this does not change the swap (si/so) or block (bi/bo)
 fields.
 .TP
+\fB\-w\fR, \fB\-\-wide\fR
+Wide output mode (useful for systems with higher amount of memory,
+where the default output mode suffers from unwanted column breakage).
+The output is wider than 80 characters per line.
+.TP
 \fB\-V\fR, \fB\-\-version\fR
 Display version information and exit.
 .TP

diff --git a/vmstat.c b/vmstat.c
index c8ccc876da39a5fcac021a1b35a2b47146aa65fc..87fa95a35d7c570f3b60e6efa1bdaf787ca3f27e 100644 (file)
--- a/vmstat.c
+++ b/vmstat.c
@@ -72,6 +72,9 @@ static int statMode = VMSTAT;
 /* "-a" means "show active/inactive" */
 static int a_option;
 
+/* "-w" means "wide output" */
+static int w_option;
+
 static unsigned sleep_time = 1;
 static int infinite_updates = 0;
 static unsigned long num_updates;
@@ -96,6 +99,7 @@ static void __attribute__ ((__noreturn__))
        fputs(_(" -D, --disk-sum         summarize disk statistics\n"), out);
        fputs(_(" -p, --partition <dev>  partition specific statistics\n"), out);
        fputs(_(" -S, --unit <char>      define display unit\n"), out);
+       fputs(_(" -w, --wide             wide output\n"), out);
        fputs(USAGE_SEPARATOR, out);
        fputs(USAGE_HELP, out);
        fputs(USAGE_VERSION, out);
@@ -179,9 +183,20 @@ static void new_header(void)
        /* Translation Hint: Translating folloging header & fields
         * that follow (marked with max x chars) might not work,
         * unless manual page is translated as well.  */
-       printf(_("procs -----------memory---------- ---swap-- -----io---- -system-- ----cpu----\n"));
-       printf
-           ("%2s %2s %6s %6s %6s %6s %4s %4s %5s %5s %4s %4s %2s %2s %2s %2s\n",
+
+       const char header[] =
+           "procs -----------memory---------- ---swap-- -----io---- -system-- ----cpu----\n";
+       const char wide_header[] =
+           "procs ---------------memory-------------- ---swap-- -----io---- -system-- ----cpu----\n";
+
+       const char format[] =
+           "%2s %2s %6s %6s %6s %6s %4s %4s %5s %5s %4s %4s %2s %2s %2s %2s\n";
+       const char wide_format[] =
+           "%2s %2s %8s %8s %8s %8s %4s %4s %5s %5s %4s %4s %2s %2s %2s %2s\n";
+
+       printf(w_option ? _(wide_header) : _(header));
+       printf(
+           w_option ? wide_format : format,
            /* Translation Hint: max 2 chars */
             _("r"),
            /* Translation Hint: max 2 chars */
@@ -231,6 +246,9 @@ static void new_format(void)
 {
        const char format[] =
            "%2u %2u %6lu %6lu %6lu %6lu %4u %4u %5u %5u %4u %4u %2u %2u %2u %2u\n";
+       const char wide_format[] =
+           "%2u %2u %8lu %8lu %8lu %8lu %4u %4u %5u %5u %4u %4u %2u %2u %2u %2u\n";
+
        unsigned int tog = 0;   /* toggle switch for cleaner code */
        unsigned int i;
        unsigned int hz = Hertz;
@@ -260,7 +278,7 @@ static void new_format(void)
        Div = duse + dsys + didl + diow + dstl;
        if (!Div) Div = 1, didl = 1;
        divo2 = Div / 2UL;
-       printf(format,
+       printf(w_option ? wide_format : format,
               running, blocked,
               unitConvert(kb_swap_used), unitConvert(kb_main_free),
               unitConvert(a_option?kb_inactive:kb_main_buffers),
@@ -315,7 +333,7 @@ static void new_format(void)
                Div = duse + dsys + didl + diow + dstl;
                if (!Div) Div = 1, didl = 1;
                divo2 = Div / 2UL;
-               printf(format,
+               printf(w_option ? wide_format : format,
                       running,
                       blocked,
                       unitConvert(kb_swap_used),unitConvert(kb_main_free),
@@ -719,6 +737,7 @@ int main(int argc, char *argv[])
                {"disk-sum", no_argument, NULL, 'D'},
                {"partition", required_argument, NULL, 'p'},
                {"unit", required_argument, NULL, 'S'},
+               {"wide", no_argument, NULL, 'w'},
                {"help", no_argument, NULL, 'h'},
                {"version", no_argument, NULL, 'V'},
                {NULL, 0, NULL, 0}
@@ -733,7 +752,7 @@ int main(int argc, char *argv[])
        atexit(close_stdout);
 
        while ((c =
-               getopt_long(argc, argv, "afmnsdDp:S:hV", longopts,
+               getopt_long(argc, argv, "afmnsdDp:S:whV", longopts,
                            NULL)) != EOF)
                switch (c) {
                case 'V':
@@ -796,6 +815,9 @@ int main(int argc, char *argv[])
                case 's':
                        statMode |= VMSUMSTAT;
                        break;
+               case 'w':
+                       w_option = 1;
+                       break;
                default:
                        /* no other aguments defined yet. */
                        usage(stderr);