Re: consider -Wmissing-variable-declarations

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

In response to

Responses

Browse pgsql-hackers by date

  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)