At Wed, 20 Jan 2021 16:40:59 +0900, Michael Paquier <mich...@paquier.xyz> wrote in > On Wed, Jan 20, 2021 at 07:25:37AM +0100, Peter Eisentraut wrote: > > It looks like we are not getting any consensus on this approach. One > > reduced version I would consider is just the second part, so you'd write > > something like > > > > snprintf(lsnchar, sizeof(lsnchar), "%X/%X", > > LSN_FORMAT_ARGS(lsn)); > > > > This would still reduce notational complexity quite a bit but avoid any > > funny business with the format strings. > > That seems reasonable to me. So +1.
That seems in the good balance. +1, too. regards. -- Kyotaro Horiguchi NTT Open Source Software Center
