On Thu, 3 Jun 2021 at 16:17, Tom Lane <t...@sss.pgh.pa.us> wrote: > > David Rowley <dgrowle...@gmail.com> 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
