Re: Abbreviated keys for Numeric

>>>>> "Peter" == Peter Geoghegan <pg@heroku.com> writes:
Peter> You still pointlessly check memtupcount here:
Peter> + if (memtupcount < 10000 || nss->input_count < 10000 || !nss->estimating)Peter> +     return false;

It's in a register; the test is free.
Peter> This cast to void is unnecessary:
Peter> + (void) ssup;

It's an explicit statement that the parameter is otherwise unused.
Maybe that compiler warning isn't usually on by default, but I
personally regard it as good style to be explicit about it.
Peter> Please try and at least consider my feedback. I don't expect youPeter> to do exactly what I ask, but I also
don'texpect you toPeter> blithely ignore it.
 

You should really stop digging this hole deeper.
>> The INT64_MIN/MAX changes should be committed fairly soon. (I>> haven't posted a patch for TRACE_SORT)
Peter> I wouldn't assume that.

Oh ye of little faith. I would not have said that had I not already been
informed of it by a committer, and indeed it is now committed.

-- 
Andrew (irc:RhodiumToad)