On Tue, Mar 04, 2025 at 08:22:02PM +0200, Heikki Linnakangas wrote: > To make things less confusing, the attached patch renames all the functions > that are part of the overall signal/interrupt handling system but are *not* > executed in a signal handler to e.g. ProcessSomething(), rather than > HandleSomething().
Am I understanding correctly that your plan is to keep the "Handle" prefix for functions that do run in signal handlers (e.g., HandleRecoveryConflictInterrupt())? I don't know how consistent the code is about that, but it might be nice to establish stricter guidelines for those, too. > Any objections? No objections here. My only concern is that this might break some third-party code, especially code that uses interrupt.h. I'm not sure it's worth adding backward-compatibility macros, though. -- nathan
