On Thu, May 07, 2026 at 11:03:43AM +0800, Kevin J. McCarthy wrote: > Mutt's general policy is to not pass NULL strings around, instead it wraps > parameters that might be NULL inside the NONULL() macro.
So I just finally looked at what NONULL does... $ grep '#.*define.*NONULL' -r . ./lib.h:# define NONULL(x) x?x:"" FWIW, I think this is bad practice. If the string was NULL it's probably a bug, and this hides the bug. I think it would generally be far better to check for NULL, write an error message/log, and abort. There may be exceptions to that but if so, they should be called out explicitly as exceptional. -- Derek D. Martin http://www.pizzashack.org/ GPG Key ID: 0xDFBEAD02 -=-=-=-=- This message is posted from an invalid address. Replying to it will result in undeliverable mail due to spam prevention. Sorry for the inconvenience.
