Gurjeet Singh <singh.gurj...@gmail.com> writes: > I noticed that the savepointLevel member of TransactionStateData struct is > initialized to 0 from TopTransactionStateData, and never incremented or > decremented afterwards.
> Since this is a file-local struct I think we can simply get rid of all > usages of this without any risk. ISTM you have detected a bug, not just dead code that should be removed. Surely those tests that throw error on savepointLevel change were meant to do something important? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers