Merge branch 'sb/wt-status-cleanup' into maint
authorJunio C Hamano <gitster@pobox.com>
Tue, 28 Mar 2017 20:52:26 +0000 (13:52 -0700)
committerJunio C Hamano <gitster@pobox.com>
Tue, 28 Mar 2017 20:52:26 +0000 (13:52 -0700)
Code clean-up.

* sb/wt-status-cleanup:
wt-status: simplify by using for_each_string_list_item

1  2 
wt-status.c
diff --combined wt-status.c
index 234e77a6d6d5cead70ff72e984c11bb7621faad6,a52d342695995d87b57a8840ae47c2a1bba92c82..4bb46781c8a7e580d4a8f6ae0dd48f8478b2d01d
@@@ -1730,14 -1730,12 +1730,14 @@@ static void wt_shortstatus_print_tracki
                return;
        branch_name = s->branch;
  
 +#define LABEL(string) (s->no_gettext ? (string) : _(string))
 +
        if (s->is_initial)
 -              color_fprintf(s->fp, header_color, _("Initial commit on "));
 +              color_fprintf(s->fp, header_color, LABEL(N_("Initial commit on ")));
  
        if (!strcmp(s->branch, "HEAD")) {
                color_fprintf(s->fp, color(WT_STATUS_NOBRANCH, s), "%s",
 -                            _("HEAD (no branch)"));
 +                            LABEL(N_("HEAD (no branch)")));
                goto conclude;
        }
  
        if (!upstream_is_gone && !num_ours && !num_theirs)
                goto conclude;
  
 -#define LABEL(string) (s->no_gettext ? (string) : _(string))
 -
        color_fprintf(s->fp, header_color, " [");
        if (upstream_is_gone) {
                color_fprintf(s->fp, header_color, LABEL(N_("gone")));
  
  static void wt_shortstatus_print(struct wt_status *s)
  {
-       int i;
+       struct string_list_item *it;
  
        if (s->show_branch)
                wt_shortstatus_print_tracking(s);
  
-       for (i = 0; i < s->change.nr; i++) {
-               struct wt_status_change_data *d;
-               struct string_list_item *it;
+       for_each_string_list_item(it, &s->change) {
+               struct wt_status_change_data *d = it->util;
  
-               it = &(s->change.items[i]);
-               d = it->util;
                if (d->stagemask)
                        wt_shortstatus_unmerged(it, s);
                else
                        wt_shortstatus_status(it, s);
        }
-       for (i = 0; i < s->untracked.nr; i++) {
-               struct string_list_item *it;
-               it = &(s->untracked.items[i]);
+       for_each_string_list_item(it, &s->untracked)
                wt_shortstatus_other(it, s, "??");
-       }
-       for (i = 0; i < s->ignored.nr; i++) {
-               struct string_list_item *it;
  
-               it = &(s->ignored.items[i]);
+       for_each_string_list_item(it, &s->ignored)
                wt_shortstatus_other(it, s, "!!");
-       }
  }
  
  static void wt_porcelain_print(struct wt_status *s)