From: Tom Lane Date: Sat, 7 Apr 2012 21:48:40 +0000 (-0400) Subject: Further adjustment of comment about qsort_tuple. X-Git-Tag: REL9_2_BETA1~169 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=95b9c333b25463283ba07c5ba7dbe79b4e8b4480;p=postgresql Further adjustment of comment about qsort_tuple. --- diff --git a/src/backend/utils/sort/tuplesort.c b/src/backend/utils/sort/tuplesort.c index 95489ac3f3..89698181db 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"