rangetypes_spgist.c contains a call to detoast a pointer which was just returned from a detoast operation. This seems to do no harm, but is to my mind a sloppy thinko. Fix attached.
tsgistidx.c contains a detoast of a pointer where detoast of a datum was likely intended. Fix attached. tsrank.c contains some arguably correct casts which I found slightly less correct than an alternative that I've attached. I'm pretty indifferent on this one, and just as happy if it is not included. Mark Dilger
tidy.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers