diff --stat: move the "total count" logic to the last loop

The diffstat generation logic, with --stat-count limit, is
implemented as three loops.

- The first counts the width necessary to show stats up to
specified number of entries, and notes up to how many entries in
the data we need to iterate to show the graph;

- The second iterates that many times to draw the graph, adjusts
the number of "total modified files", and counts the total
added/deleted lines for the part that was shown in the graph;

- The third iterates over the remainder and only does the part to
count "total added/deleted lines" and to adjust "total modified
files" without drawing anything.

Move the logic to count added/deleted lines and modified files from
the second loop to the third loop.

This incidentally fixes a bug.  The third loop was not filtering
binary changes (counted in bytes) from the total added/deleted as it
should.  The second loop implemented this correctly, so if a binary
change appeared earlier than the --stat-count cutoff, the code
counted number of added/deleted lines correctly, but if it appeared
beyond the cutoff, the number of lines would have mixed with the
byte count in the buggy third loop.

Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/diff.c b/diff.c
index e4e70e55fe27e6444b030aec07b43cb0ee603683..4105260122878a81994077e1552dcc305f501037 100644 (file)
--- a/diff.c
+++ b/diff.c
@@ -1498,7 +1498,7 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
                if (max_change < change)
                        max_change = change;
        }
                if (max_change < change)
                        max_change = change;
        }

-       count = i; /* min(count, data->nr) */
+       count = i; /* where we can stop scanning in data->files[] */

 
        /*
         * We have width = stat_width or term_columns() columns total.
 
        /*
         * We have width = stat_width or term_columns() columns total.


@@ -1592,10 +1592,9 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
                uintmax_t deleted = file->deleted;
                int name_len;
 
                uintmax_t deleted = file->deleted;
                int name_len;
 

-               if (!file->is_interesting && (added + deleted == 0)) {
-                       total_files--;
+               if (!file->is_interesting && (added + deleted == 0))

                        continue;
                        continue;

-               }
+

                /*
                 * "scale" the filename
                 */
                /*
                 * "scale" the filename
                 */


@@ -1640,8 +1639,6 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
                 */
                add = added;
                del = deleted;
                 */
                add = added;
                del = deleted;

-               adds += add;
-               dels += del;

 
                if (graph_width <= max_change) {
                        int total = add + del;
 
                if (graph_width <= max_change) {
                        int total = add + del;


@@ -1667,7 +1664,8 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
                show_graph(options->file, '-', del, del_c, reset);
                fprintf(options->file, "\n");
        }
                show_graph(options->file, '-', del, del_c, reset);
                fprintf(options->file, "\n");
        }

-       for (i = count; i < data->nr; i++) {
+
+       for (i = 0; i < data->nr; i++) {

                struct diffstat_file *file = data->files[i];
                uintmax_t added = file->added;
                uintmax_t deleted = file->deleted;
                struct diffstat_file *file = data->files[i];
                uintmax_t added = file->added;
                uintmax_t deleted = file->deleted;


@@ -1675,8 +1673,13 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
                        total_files--;
                        continue;
                }
                        total_files--;
                        continue;
                }

-               adds += added;
-               dels += deleted;
+
+               if (!file->is_binary && !file->is_unmerged) {
+                       adds += added;
+                       dels += deleted;
+               }
+               if (i < count)
+                       continue;

                if (!extra_shown)
                        fprintf(options->file, "%s ...\n", line_prefix);
                extra_shown = 1;
                if (!extra_shown)
                        fprintf(options->file, "%s ...\n", line_prefix);
                extra_shown = 1;

diff --git a/t/t4049-diff-stat-count.sh b/t/t4049-diff-stat-count.sh
index e212b11867c87661afb3a3ac85a6fdaf7a834225..70ee0736813fd4db389f996080bfdb28515ffc8b 100755 (executable)
--- a/t/t4049-diff-stat-count.sh
+++ b/t/t4049-diff-stat-count.sh
@@ -28,7 +28,7 @@ test_expect_success 'limit output to 2 (simple)' '
        test_i18ncmp expect actual
 '
 
        test_i18ncmp expect actual
 '
 

-test_expect_failure 'binary changes do not count in lines' '
+test_expect_success 'binary changes do not count in lines' '

        git reset --hard &&
        chmod +x c d &&
        echo a >a &&
        git reset --hard &&
        chmod +x c d &&
        echo a >a &&