Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1ltgc4-0004Sc-S7 for pgsql-hackers@arkaria.postgresql.org; Thu, 17 Jun 2021 01:14:32 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1ltgc3-0004hl-ES for pgsql-hackers@arkaria.postgresql.org; Thu, 17 Jun 2021 01:14:31 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1ltgc3-0004hd-6i for pgsql-hackers@lists.postgresql.org; Thu, 17 Jun 2021 01:14:31 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1ltgc1-0006Ii-AR for pgsql-hackers@postgresql.org; Thu, 17 Jun 2021 01:14:30 +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 15H1EPKx930426; Wed, 16 Jun 2021 21:14:25 -0400 From: Tom Lane To: Thomas Munro cc: Zhihong Yu , Andres Freund , Daniel Gustafsson , pgsql-hackers Subject: Re: A qsort template In-reply-to: References: <0F9ECBA9-1DF2-4C38-BB58-B17EEE6C7B3A@yesql.se> <20210311185847.zwb2rf3n4wzelkba@alap3.anarazel.de> <870660.1623886818@sss.pgh.pa.us> Comments: In-reply-to Thomas Munro message dated "Thu, 17 Jun 2021 13:07:38 +1200" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <930424.1623892465.1@sss.pgh.pa.us> Date: Wed, 16 Jun 2021 21:14:25 -0400 Message-ID: <930425.1623892465@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Thomas Munro writes: > Perhaps one day we could add a > secondary file, not updated by that mechanism, that holds a manually > maintained list for cases like this. Yeah, the comments in pgindent already speculate about that. For now, those include and exclude lists are short enough that keeping them inside the script seems a lot easier than building tooling to get them from somewhere else. The big problem in my mind, which would not be alleviated in the slightest by having a separate file, is that it'd be easy to miss removing entries if they ever become obsolete. regards, tom lane