Re: use of SPI by postgresImportForeignStatistics - Mailing list pgsql-hackers

On Fri, Jul 10, 2026 at 4:40 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> On Fri, Jul 10, 2026 at 2:37 PM Fujii Masao <masao.fujii@gmail.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.

Best regards,
Etsuro Fujita