Thread (19 messages) 19 messages, 3 authors, 2025-03-19

Re: [PATCH v3 3/3] treewide: replace assert() with ASSERT() in special cases

From: Taylor Blau <hidden>
Date: 2025-03-19 22:27:41

On Wed, Mar 19, 2025 at 04:22:58PM +0000, Elijah Newren via GitGitGadget wrote:
From: Elijah Newren <redacted>

When the compiler/linker cannot verify that an assert() invocation is
free of side effects for us (e.g. because the assertion includes some
kind of function call), replace the use of assert() with ASSERT().
As a nice side-benefit, since our new ASSERT() macro naturally has the
same number of characters as a bog-standard assert(), we don't have to
realign any multi-line assertions below ;-).

Thanks,
Taylor
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help