From 95b9c333b25463283ba07c5ba7dbe79b4e8b4480 Mon Sep 17 00:00:00 2001 From: Tom Lane <tgl@sss.pgh.pa.us> Date: Sat, 7 Apr 2012 17:48:40 -0400 Subject: [PATCH] Further adjustment of comment about qsort_tuple. --- src/backend/utils/sort/tuplesort.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/backend/utils/sort/tuplesort.c b/src/backend/utils/sort/tuplesort.c index 95489ac3f30..89698181dbf 100644 --- a/src/backend/utils/sort/tuplesort.c +++ b/src/backend/utils/sort/tuplesort.c @@ -346,7 +346,7 @@ struct Tuplesortstate /* * This variable is shared by the single-key MinimalTuple case and the - * Datum case. Otherwise it's NULL. + * Datum case (which both use qsort_ssup()). Otherwise it's NULL. */ SortSupport onlyKey; @@ -500,8 +500,11 @@ static void reversedirection_datum(Tuplesortstate *state); static void free_sort_tuple(Tuplesortstate *state, SortTuple *stup); /* - * Special versions of qsort just for SortTuple objects. We have one for the - * single-key case (qsort_ssup) and one for multi-key cases (qsort_tuple). + * Special versions of qsort just for SortTuple objects. qsort_tuple() sorts + * any variant of SortTuples, using the appropriate comparetup function. + * qsort_ssup() is specialized for the case where the comparetup function + * reduces to ApplySortComparator(), that is single-key MinimalTuple sorts + * and Datum sorts. */ #include "qsort_tuple.c" -- GitLab