]> granicus.if.org Git - mutt/commit
Fix non-threaded $sort_aux "reverse-" sorting.
authorKevin McCarthy <kevin@8t8.us>
Wed, 26 Sep 2018 02:03:56 +0000 (19:03 -0700)
committerKevin McCarthy <kevin@8t8.us>
Wed, 26 Sep 2018 02:03:56 +0000 (19:03 -0700)
commit999d85c69e02ad3560f1c9928ab526c3dad69ab3
tree99c5e5df878ee3bc58b9e765e50c984f38321b40
parentf8196e20f141b4fd3a3ed5e6e809d016d592a70f
Fix non-threaded $sort_aux "reverse-" sorting.

The secondary sort was looking at (Sort & SORT_REVERSE) instead
of (SortAux & SORT_REVERSE), so wasn't even performing the reverse
based on the correct flag.

Additionally, afterwards, the primary sort was improperly applying a
reverse when the secondary sort returned non-zero.

Change SORTCODE() to look at SortAux when we are inside a secondary
sort.

Change AUXSORT() to return the result if the secondary sort returns
non-zero.  It is ugly to put a return inside the macro, but the check
for non-zero needs to be performed inside the AUXSORT if branch.

If the secondary sort returns 0, then the primary sort can still
compare index and apply a reverse as needed.
init.h
sort.c