On Thu, Nov 18, 2021 at 11:23:17AM +0900, Fujii Masao wrote: > Yes, that's possible. But isn't it uglier to make ExecuteRecoveryCommand() > have > the map of command name and wait event? So I feel inclined to avoid adding > something like the following code into the function... Thought?
FWIW, I find cleaner, and less bug-prone, the approach taken by Fujii-san's patch to have the wait event set as an argument of the function rather than trying to guess it from the command data. -- Michael
signature.asc
Description: PGP signature