> On 25 Sep 2025, at 09:23, Hayato Kuroda (Fujitsu) <[email protected]> > wrote: > > Dear hackers, > >> While reviewing others, I found $SUBJECT. > > I found another cleanup point related with this. In CreateCheckPoint(): > > ``` > #ifdef USE_INJECTION_POINTS > INJECTION_POINT("checkpoint-before-old-wal-removal"); > #endif > ``` > > Here USE_INJECTION_POINTS check is not needed. If the feature is disabled, > the macro function would be ((void) name). IIUC, we are using the macro if > if-branch > exists.
+1, that's not needed (and not used elsewhere in the code either). -- Daniel Gustafsson
