The bounded heap sorting status flag is set twice in sort_bounded_heap() and tuplesort_performsort(). This patch helps remove one of them.
Revisiting this patch I think maybe it's better to remove the setting of Tuplesort status from tuplesort_performsort() for the TSS_BOUNDED case. Thus we keep the heap manipulation routines, make_bounded_heap and sort_bounded_heap, consistent in that they update their status accordingly inside the function.
Also, would you please add it to the CF to not lose track of it?
