cpluspluscheck failure

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: cpluspluscheck failure
Date: 2022-01-12 15:24:34
Message-ID: 491a3724-7c20-3ae1-8653-8dce986c379b@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


While testing a buildfarm module to automate running headerscheck and
cpluspluscheck, I encountered a bunch of errors like this:

Jan 12 09:35:57 In file included from
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/port/atomics.h:70,
Jan 12 09:35:57                  from
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/storage/lwlock.h:21,
Jan 12 09:35:57                  from
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/storage/lock.h:23,
Jan 12 09:35:57                  from
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/storage/proc.h:21,
Jan 12 09:35:57                  from
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/storage/shm_mq.h:18,
Jan 12 09:35:57                  from
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/libpq/pqmq.h:17,
Jan 12 09:35:57                  from /tmp/cpluspluscheck.16q7jo/test.cpp:3:
Jan 12 09:35:57
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/port/atomics/arch-x86.h:
In function ‘bool pg_atomic_test_set_flag_impl(volatile pg_atomic_flag*)’:
Jan 12 09:35:57
/home/andrew/bf/root/HEAD/pgsql.build/../pgsql/src/include/port/atomics/arch-x86.h:143:23:
warning: ISO C++17 does not allow ‘register’ storage class specifier
[-Wregister]
Jan 12 09:35:57   143 |         register char _res = 1;
Jan 12 09:35:57       |                       ^~~~

there are similar complaints about s_lock.h.

The compiler is: g++ (GCC) 11.2.1 20210728 (Red Hat 11.2.1-1)

Do we need to add -Wnoregister to the CXXFLAGS?

cheers

andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com

Browse pgsql-hackers by date

  From Date Subject
Next Message John Naylor 2022-01-12 15:42:06 Re: do only critical work during single-user vacuum?
Previous Message Jelte Fennema 2022-01-12 15:22:18 Add non-blocking version of PQcancel