> On 25 Sep 2025, at 09:23, Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com> 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