On Fri, Jul 10, 2026 at 4:40 PM Etsuro Fujita <[email protected]> wrote: > On Fri, Jul 10, 2026 at 2:37 PM Fujii Masao <[email protected]> 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
