On Thu, 3 Jun 2021 at 16:17, Tom Lane <[email protected]> wrote: > > David Rowley <[email protected]> writes: > > There are quite a few other places in that file that should be using > > DatumGetCString() instead of DatumGetPointer(). > > Should we fix those too for PG14? > > +1. I'm surprised we are not getting compiler warnings.
I pushed a fix for this. I did happen to find one other in mcv.c which dates back to 2019. I was wondering if we should bother with that one since it's already out there in PG13. David
use_OutputFunctionCall_in_mcv.c.patch
Description: Binary data
