pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Nathan Bossart <nathandbossart@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: Simplifying our Trap/Assert infrastructure
Date: Fri, 28 Oct 2022 09:36:23 +0200
Message-ID: <572ed6f8-a137-0a52-a35e-a48df95af6ed@enterprisedb.com> (raw)
In-Reply-To: <Y1oyCXhg/6Odn6L/@paquier.xyz>
References: <3928703.1665345117@sss.pgh.pa.us>
	<20221009210148.GA900071@nathanxps13>
	<20221012183620.GA1447761@nathanxps13>
	<fe8a3770-1256-cf96-f3fa-db526fd09568@enterprisedb.com>
	<Y1oyCXhg/6Odn6L/@paquier.xyz>

On 27.10.22 09:23, Michael Paquier wrote:
> Agreed, even if extensions could use these, it looks like any
> out-of-core code using what's removed here would also gain in clarity.
> This is logically fine (except for an indentation blip in
> miscadmin.h?), so I have marked this entry as ready for committer.

committed

> Side note, rather unrelated to what's proposed here: would it be worth
> extending AssertPointerAlignment() for the frontend code?

Would there be a use for that?  It's currently only used in the atomics 
code.






view thread (13+ messages)  latest in thread

Message-ID: <572ed6f8-a137-0a52-a35e-a48df95af6ed@enterprisedb.com>
Permalink:  ../572ed6f8-a137-0a52-a35e-a48df95af6ed@enterprisedb.com/
Also on:    postgresql.org/message-id/572ed6f8-a137-0a52-a35e-a48df95af6ed@enterprisedb.com

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: peter.eisentraut@enterprisedb.com, michael@paquier.xyz, nathandbossart@gmail.com, tgl@sss.pgh.pa.us, pgsql-hackers@lists.postgresql.org
  Subject: Re: Simplifying our Trap/Assert infrastructure
  In-Reply-To: <572ed6f8-a137-0a52-a35e-a48df95af6ed@enterprisedb.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox