[PATCH 08/25] list-files: add --[no-]column, -C and -1

[Nguyễn Thái Ngọc Duy]

Signed-off-by: Nguyễn Thái Ngọc Duy <pclo...@gmail.com>
---
 builtin/list-files.c  | 22 +++++++++++++++++++++-
 t/t7013-list-files.sh | 27 +++++++++++++++++++++++++++
 2 files changed, 48 insertions(+), 1 deletion(-)

diff --git a/builtin/list-files.c b/builtin/list-files.c
index ec9ca17..8913770 100644
--- a/builtin/list-files.c
+++ b/builtin/list-files.c
@@ -4,6 +4,7 @@
 #include "pathspec.h"
 #include "dir.h"
 #include "quote.h"
+#include "column.h"
 
 enum item_type {
        FROM_INDEX
@@ -26,6 +27,7 @@ static struct pathspec pathspec;
 static const char *prefix;
 static int prefix_length;
 static int show_tag = -1;
+static unsigned int colopts;
 
 static const char * const ls_usage[] = {
        N_("git list-files [options] [<pathspec>...]"),
@@ -34,6 +36,9 @@ static const char * const ls_usage[] = {
 
 struct option ls_options[] = {
        OPT_BOOL(0, "tag", &show_tag, N_("show tags")),
+       OPT_COLUMN('C', "column", &colopts, N_("show files in columns")),
+       OPT_SET_INT('1', NULL, &colopts,
+                   N_("shortcut for --no-column"), COL_PARSEOPT),
        OPT_END()
 };
 
@@ -114,6 +119,7 @@ static void quote_item(struct strbuf *out, const struct 
item *item)
 static void display(const struct item_list *result)
 {
        struct strbuf quoted = STRBUF_INIT;
+       struct string_list s = STRING_LIST_INIT_DUP;
        int i;
 
        if (!prefix_length)
@@ -129,13 +135,26 @@ static void display(const struct item_list *result)
                        strbuf_addch(&quoted, ' ');
                }
                quote_item(&quoted, item);
-               printf("%s\n", quoted.buf);
+               if (column_active(colopts))
+                       string_list_append(&s, quoted.buf);
+               else
+                       printf("%s\n", quoted.buf);
+       }
+
+       if (column_active(colopts)) {
+               struct column_options copts;
+               memset(&copts, 0, sizeof(copts));
+               copts.padding = 2;
+               print_columns(&s, colopts, &copts);
+               string_list_clear(&s, 0);
        }
        strbuf_release(&quoted);
 }
 
 static int ls_config(const char *var, const char *value, void *cb)
 {
+       if (starts_with(var, "column."))
+               return git_column_config(var, value, "listfiles", &colopts);
        return git_default_config(var, value, cb);
 }
 
@@ -163,6 +182,7 @@ int cmd_list_files(int argc, const char **argv, const char 
*cmd_prefix)
                       cmd_prefix, argv);
        pathspec.max_depth = 0;
        pathspec.recursive = 1;
+       finalize_colopts(&colopts, -1);
 
        refresh_index(&the_index, REFRESH_QUIET | REFRESH_UNMERGED,
                      &pathspec, NULL, NULL);
diff --git a/t/t7013-list-files.sh b/t/t7013-list-files.sh
index a5e555d..b43245c 100755
--- a/t/t7013-list-files.sh
+++ b/t/t7013-list-files.sh
@@ -23,6 +23,33 @@ test_expect_success 'list-files from index' '
        test_cmp expect actual
 '
 
+test_expect_success 'column output' '
+       COLUMNS=20 git list-files --column=always >actual &&
+       cat >expected <<-\EOF &&
+       a        sa/a
+       b        sa/sb/b
+       c        sc/c
+       EOF
+       test_cmp expected actual &&
+       COLUMNS=20 git -c column.listfiles=always list-files >actual &&
+       cat >expected <<-\EOF &&
+       a        sa/a
+       b        sa/sb/b
+       c        sc/c
+       EOF
+       test_cmp expected actual &&
+       COLUMNS=20 git -c column.listfiles=always list-files -1 >actual &&
+       cat >expected <<-\EOF &&
+       a
+       b
+       c
+       sa/a
+       sa/sb/b
+       sc/c
+       EOF
+       test_cmp expected actual
+'
+
 test_expect_success 'list-files selectively from index' '
        git list-files "*a" >actual &&
        cat >expect <<-\EOF &&
-- 
2.3.0.rc1.137.g477eb31

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html