On Mon, Jul 01, 2024 at 06:42:46AM +0000, Bertrand Drouvot wrote: > While working on a rebase for [1] due to 0cecc908e97, I noticed that > CheckRelationLockedByMe() and CheckRelationOidLockedByMe() are used only in > assertions. > > I think it would make sense to declare / define those functions only for > assert enabled build: please find attached a tiny patch doing so. > > Thoughts?
Not convinced that's a good idea. What about out-of-core code that may use these routines for runtime checks in non-assert builds? -- Michael
signature.asc
Description: PGP signature
