On Thu, 20 Dec 2018 at 05:44, Tom Lane <t...@sss.pgh.pa.us> wrote: > I don't think this is quite bulletproof against overflow, especially > in view of the rather careless mixing of int32 and uint32 variables > that exists here. The easiest way to make it bulletproof is to add > an explicit test, so I did that and pushed it.
Thanks for tidying that up and for pushing. -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services