Re: Move catalog toast table and index declarations

On 2020-10-27 13:12, John Naylor wrote:
> There's nothing wrong; it's just a minor point of consistency. For the 
> first part, I mean defined symbols in this file that are invisible to 
> the C compiler are written
> 
> #define SOMETHING()
> 
> If some are written
> 
> #define SOMETHING() extern int no_such_variable
> 
> I imagine some future reader will wonder why there's a difference.

The difference is that CATALOG() is followed in actual use by something like

     { ... } FormData_pg_attribute;

so it becomes a valid C statement.  For DECLARE_INDEX() etc., we need to 
do something else to make it valid.  I guess this could be explained in 
more detail (as I'm attempting in this email), but this isn't materially 
changed by this patch.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services