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.94.2) (envelope-from ) id 1qlEWf-001r98-2N for pgsql-hackers@arkaria.postgresql.org; Tue, 26 Sep 2023 20:19:21 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1qlEWc-00Elxb-Oj for pgsql-hackers@arkaria.postgresql.org; Tue, 26 Sep 2023 20:19:18 +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.94.2) (envelope-from ) id 1qlEWc-00ElxL-Dj for pgsql-hackers@lists.postgresql.org; Tue, 26 Sep 2023 20:19:18 +0000 Received: from mail.thelabyrinth.net ([45.56.70.56]) by magus.postgresql.org with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1qlEWZ-007kaY-F7 for pgsql-hackers@postgresql.org; Tue, 26 Sep 2023 20:19:17 +0000 Received: from [10.5.0.2] (unknown [193.43.135.119]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: dsteele) by mail.thelabyrinth.net (Postfix) with ESMTPSA id F17A354677; Tue, 26 Sep 2023 20:19:12 +0000 (UTC) Message-ID: Date: Tue, 26 Sep 2023 16:19:12 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Subject: Re: Add const qualifiers To: Peter Eisentraut , Pg Hackers References: <669a035c-d23d-2f38-7ff0-0cb93e01d610@pgmasters.net> <96cb71e3-13d3-543e-fe3d-9c7034fc6574@eisentraut.org> Content-Language: en-US From: David Steele In-Reply-To: <96cb71e3-13d3-543e-fe3d-9c7034fc6574@eisentraut.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On 9/26/23 06:34, Peter Eisentraut wrote: > On 09.09.23 21:03, David Steele wrote: >> On 9/1/23 11:39, David Steele wrote: >>> Hackers, >>> >>> I noticed that there was a mismatch between the const qualifiers for >>> excludeDirContents in src/backend/backup/backup/basebackup.c and >>> src/bin/pg_rewind/file_map.c and that led me to use ^static >>> const.*\*.*= to do a quick search for similar cases. >>> >>> I think at the least we should make excludeDirContents match, but the >>> rest of the changes seem like a good idea as well. >> >> Added to 2023-11 CF. > > committed Thank you, Peter!