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 1wE7If-003gp2-0c for pgsql-hackers@arkaria.postgresql.org; Sat, 18 Apr 2026 15:09:37 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.96) (envelope-from ) id 1wE7Ie-00DAwz-1I for pgsql-hackers@arkaria.postgresql.org; Sat, 18 Apr 2026 15:09:36 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wE7Ie-00DAwr-0P for pgsql-hackers@lists.postgresql.org; Sat, 18 Apr 2026 15:09:36 +0000 Received: from sss.pgh.pa.us ([68.162.161.243]) by makus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.98.2) (envelope-from ) id 1wE7Ic-00000001dZn-07Bw for pgsql-hackers@lists.postgresql.org; Sat, 18 Apr 2026 15:09:35 +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 63IF9VdT3432491; Sat, 18 Apr 2026 11:09:31 -0400 From: Tom Lane To: David Rowley cc: Peter Eisentraut , Chao Li , PostgreSQL Developers Subject: Re: Add bms_offset_members() function for bitshifting Bitmapsets In-reply-to: References: <278B9FE3-F349-4494-99C5-483105C1C999@gmail.com> <1900289.1776212948@sss.pgh.pa.us> <1913188.1776220247@sss.pgh.pa.us> Comments: In-reply-to David Rowley message dated "Sat, 18 Apr 2026 19:53:40 +1200" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <3432489.1776524971.1@sss.pgh.pa.us> Date: Sat, 18 Apr 2026 11:09:31 -0400 Message-ID: <3432490.1776524971@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk David Rowley writes: > On Thu, 16 Apr 2026 at 07:17, Peter Eisentraut wrote: >> Depending on what you end up doing, maybe a sprinkling of pg_nodiscard >> could be appropriate. > Yeah maybe. It wouldn't do any harm, at least. > I didn't do that in the patch I just sent as it felt like something we > should do or not do for all the bitmapset functions it's relevant for. Agreed, seems like it should be a separate patch. regards, tom lane