Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wA7Ww-0025By-2k for pgsql-hackers@arkaria.postgresql.org; Tue, 07 Apr 2026 14:35:50 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.96) (envelope-from ) id 1wA7Wv-000mJL-17 for pgsql-hackers@arkaria.postgresql.org; Tue, 07 Apr 2026 14:35:49 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wA7Wv-000mJD-0D for pgsql-hackers@lists.postgresql.org; Tue, 07 Apr 2026 14:35:49 +0000 Received: from sss.pgh.pa.us ([68.162.161.243]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.98.2) (envelope-from ) id 1wA7Wt-000000017r7-0PPk for pgsql-hackers@postgresql.org; Tue, 07 Apr 2026 14:35:49 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.15.2/8.15.2) with ESMTP id 637EZRhs2042831; Tue, 7 Apr 2026 10:35:27 -0400 From: Tom Lane To: Peter Eisentraut cc: Jelte Fennema-Nio , pgsql-hackers Subject: Re: enable fallthrough warnings on clang In-reply-to: <94a2bc01-e1bf-4e07-ba58-18dc5aacc0e9@eisentraut.org> References: <76a8efcd-925a-4eaf-bdd1-d972cd1a32ff@eisentraut.org> <955ddaa3-d6e8-4d87-a368-ef9f25960d4e@eisentraut.org> <1914806.1775511109@sss.pgh.pa.us> <94a2bc01-e1bf-4e07-ba58-18dc5aacc0e9@eisentraut.org> Comments: In-reply-to Peter Eisentraut message dated "Tue, 07 Apr 2026 16:31:49 +0200" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <2042829.1775572527.1@sss.pgh.pa.us> Date: Tue, 07 Apr 2026 10:35:27 -0400 Message-ID: <2042830.1775572527@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Peter Eisentraut writes: > On 06.04.26 23:31, Tom Lane wrote: >> BF member ayu is failing said test: >> ... >> I don't know if it's worth catering to this extremely old >> clang version ... > But AFAICT, clang 5 and older are completely broken in this regard, > because they react to the presence of [[clang::fallthrough]] with a > syntax error ("error: expected expression"). Not even clang's own > example code[0] works. Hah, so we'd make it worse not better by adding [[clang::fallthrough]]. Let's leave well enough alone then. regards, tom lane