Re: use of SPI by postgresImportForeignStatistics

On Fri, Jul 10, 2026 at 8:37 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
>
> On Fri, Jul 10, 2026 at 4:40 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> > On Fri, Jul 10, 2026 at 2:37 PM Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
> > > In the committed patch, I found that the set_*_arg() helper functions
> > > in postgres_fdw.c are declared static in their prototypes, but their
> > > definitions omit the static keyword. This looks like an oversight.
> > >
> > > Attached patch adds static to the definitions for consistency and
> > > to make their intended file-local scope explicit. It also fixes a few
> > > nearby comment typos.
> >
> > Good catch! Thanks for the patch! LGTM.
>
> My compiler (Apple clang version 17.0.0 (clang-1700.4.4.1)) doesn't
> output any error/warning about that declaration. Actually, it's
> allowed? Anyway, +1 for adding it for consistency.

Maybe that's allowed. I wonder adding static to both the declaration
and the definition seems mainly a matter of consistency, readability,
and coding style.

Thanks for the review! I've pushed the patch.

Regards,

--
Fujii Masao