]> git.ipfire.org Git - thirdparty/git.git/commit - t/t3203-branch-output.sh
ref-filter: fallback on alphabetical comparison
authorKarthik Nayak <karthik.188@gmail.com>
Fri, 30 Oct 2015 08:45:28 +0000 (14:15 +0530)
committerJunio C Hamano <gitster@pobox.com>
Fri, 30 Oct 2015 20:50:52 +0000 (13:50 -0700)
commit9e468334b41c1d1fc715de177ef1f61a36c1cf01
tree5861b609930bbfe709c1f8f09bceb2b2fe570d1a
parentaa3bc55e408d4daab52239d6b80f7d4bb87f6de7
ref-filter: fallback on alphabetical comparison

In ref-filter.c the comparison of refs while sorting is handled by
cmp_ref_sorting() function. When sorting as per numerical values
(e.g. --sort=objectsize) there is no fallback comparison when both
refs hold the same value. This can cause unexpected results (i.e. the
order of listing refs with equal values cannot be pre-determined) as
pointed out by Johannes Sixt ($gmane/280117).

Hence, fallback to alphabetical comparison based on the refname
whenever the other criterion is equal.

A test in t3203 was expecting that branch-two sorts before HEAD, which
happened to be how qsort(3) on Linux sorted the array, but (1) that
outcome was not even guaranteed, and (2) once we start breaking ties
with the refname, "HEAD" should sort before "branch-two" so the
original expectation was inconsistent with the criterion we now use.

Update it to match the new world order, which we can now depend on
being stable.

Helped-by: Junio C Hamano <gitster@pobox.com>
Reported-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Karthik Nayak <Karthik.188@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
ref-filter.c
t/t3203-branch-output.sh