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.

E.g., another injection point "create-restart-point" does not have the #ifdef 
part.

Best regards,
Hayato Kuroda
FUJITSU LIMITED



Reply via email to