src/shared/format-table.c

@@ -1010,24 +1010,6 @@ int table_set_empty_string(Table *t, const char *empty) {
         return free_and_strdup(&t->empty_string, empty);
 }
 
-int table_set_display_all(Table *t) {
-        size_t allocated;
-
-        assert(t);
-
-        allocated = t->n_display_map;
-
-        if (!GREEDY_REALLOC(t->display_map, allocated, MAX(t->n_columns, allocated)))
-                return -ENOMEM;
-
-        for (size_t i = 0; i < t->n_columns; i++)
-                t->display_map[i] = i;
-
-        t->n_display_map = t->n_columns;
-
-        return 0;
-}
-
 int table_set_display(Table *t, size_t first_column, ...) {
         size_t allocated, column;
         va_list ap;
@@ -1087,34 +1069,6 @@ int table_set_sort(Table *t, size_t first_column, ...) {
         return 0;
 }
 
-int table_hide_column_from_display(Table *t, size_t column) {
-        size_t allocated, cur = 0;
-        int r;
-
-        assert(t);
-        assert(column < t->n_columns);
-
-        /* If the display map is empty, initialize it with all available columns */
-        if (!t->display_map) {
-                r = table_set_display_all(t);
-                if (r < 0)
-                        return r;
-        }
-
-        allocated = t->n_display_map;
-
-        for (size_t i = 0; i < allocated; i++) {
-                if (t->display_map[i] == column)
-                        continue;
-
-                t->display_map[cur++] = t->display_map[i];
-        }
-
-        t->n_display_map = cur;
-
-        return 0;
-}
-
 static int cell_data_compare(TableData *a, size_t index_a, TableData *b, size_t index_b) {
         assert(a);
         assert(b);

src/shared/format-table.h

@@ -101,11 +101,9 @@ void table_set_header(Table *table, bool b);
 void table_set_width(Table *t, size_t width);
 void table_set_cell_height_max(Table *t, size_t height);
 int table_set_empty_string(Table *t, const char *empty);
-int table_set_display_all(Table *t);
 int table_set_display(Table *t, size_t first_column, ...);
 int table_set_sort(Table *t, size_t first_column, ...);
 int table_set_reverse(Table *t, size_t column, bool b);
-int table_hide_column_from_display(Table *t, size_t column);
 
 int table_print(Table *t, FILE *f);
 int table_format(Table *t, char **ret);

src/systemctl/systemctl.c

@@ -400,12 +400,6 @@ static int output_units_list(const UnitInfo *unit_infos, unsigned c) {
                 return log_oom();
 
         table_set_header(table, !arg_no_legend);
-        if (arg_no_legend) {
-                /* Hide the 'glyph' column when --no-legend is requested */
-                r = table_hide_column_from_display(table, 0);
-                if (r < 0)
-                        return log_error_errno(r, "Failed to hide column: %m");
-        }
         if (arg_full)
                 table_set_width(table, 0);
 
@@ -467,9 +461,12 @@ static int output_units_list(const UnitInfo *unit_infos, unsigned c) {
 
         if (job_count == 0) {
                 /* There's no data in the JOB column, so let's hide it */
-                r = table_hide_column_from_display(table, 5);
+                /* Also, convert all number constants to size_t so va_arg()
+                 * in table_set_display() fetches a correct number of bytes from
+                 * the stack */
+                r = table_set_display(table, (size_t) 0, (size_t) 1, (size_t) 2, (size_t) 3, (size_t) 4, (size_t) 6, (size_t) -1);
                 if (r < 0)
-                        return log_error_errno(r, "Failed to hide column: %m");
+                        return log_error_errno(r, "Failed to set columns to display: %m");
         }
 
         r = table_print(table, NULL);
@@ -1983,12 +1980,6 @@ static int output_machines_list(struct machine_info *machine_infos, unsigned n)
                 return log_oom();
 
         table_set_header(table, !arg_no_legend);
-        if (arg_no_legend) {
-                /* Hide the 'glyph' column when --no-legend is requested */
-                r = table_hide_column_from_display(table, 0);
-                if (r < 0)
-                        return log_error_errno(r, "Failed to hide column: %m");
-        }
         if (arg_full)
                 table_set_width(table, 0);