Now when checking for shortening of leaf tuple is added LongValuesOK become slightly redundant. I'd propose to replace it with more legible name as LongValuesOK doesn't mean they are warranted to be ok just that we can try to shorten them? What about tryShortening, trySuffixing or can(Try)ShortenTuple?
-- Best regards, Pavel Borisov Postgres Professional: http://postgrespro.com <http://www.postgrespro.com>