From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Cc: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Andres Freund <andres(at)anarazel(dot)de> |
Subject: | Re: consider -Wmissing-variable-declarations |
Date: | 2024-07-26 09:07:25 |
Message-ID: | 6070f0ee-176f-4e78-9324-3bbb70a191f9@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I have committed all of the fixes that I had previously posted, but
before actually activating the warning option, I found another small
hiccup with the Bison files.
Before Bison 3.4, the generated parser implementation files run afoul of
-Wmissing-variable-declarations (in spite of commit ab61c40bfa2) because
declarations for yylval and possibly yylloc are missing. The generated
header files contain an extern declaration, but the implementation files
don't include the header files. Since Bison 3.4, the generated
implementation files automatically include the generated header files,
so then it works.
To make this work with older Bison versions as well, I made a patch to
include the generated header file from the .y file.
(With older Bison versions, the generated implementation file contains
effectively a copy of the header file pasted in, so including the header
file is redundant. But we know this works anyway because the core
grammar uses this arrangement already.)
Attachment | Content-Type | Size |
---|---|---|
v3-0001-Include-bison-header-files-into-implementation-fi.patch | text/plain | 4.8 KB |
v3-0002-Add-Wmissing-variable-declarations-to-the-standar.patch | text/plain | 8.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tender Wang | 2024-07-26 09:08:34 | Re: [BUG] Fix DETACH with FK pointing to a partitioned table fails |
Previous Message | Aleksander Alekseev | 2024-07-26 09:01:40 | Re: [PATCH] Add crc32(text) & crc32(bytea) |