On Fri, Mar 12, 2021 at 1:35 PM Peter Geoghegan <p...@bowt.ie> wrote: > On Fri, Mar 12, 2021 at 10:32 AM Peter Geoghegan <p...@bowt.ie> wrote: > > Thank you both, Mark and Robert. This is excellent work!
Thanks. > FYI I see these compiler warnings just now: > > pg_amcheck.c:1653:4: warning: ISO C90 forbids mixed declarations and > code [-Wdeclaration-after-statement] > 1653 | DatabaseInfo *dat = (DatabaseInfo *) > pg_malloc0(sizeof(DatabaseInfo)); > | ^~~~~~~~~~~~ > pg_amcheck.c: In function ‘compile_relation_list_one_db’: > pg_amcheck.c:2060:9: warning: variable ‘is_btree’ set but not used > [-Wunused-but-set-variable] > 2060 | bool is_btree = false; > | ^~~~~~~~ > > Looks like this 'is_btree' variable should be PG_USED_FOR_ASSERTS_ONLY. I'll commit something shortly to address these. -- Robert Haas EDB: http://www.enterprisedb.com