Re: Update the LSN format in the comment example

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Japin Li <japinli(at)hotmail(dot)com>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Update the LSN format in the comment example
Date: 2025-08-14 02:13:28
Message-ID: CAHGQGwFix8N1+480Vud0+6zf99-ChYctrJi2YMHU90gtzNJ6fA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Aug 12, 2025 at 7:36 PM Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
>
> On Tue, Aug 12, 2025 at 4:45 PM Japin Li <japinli(at)hotmail(dot)com> wrote:
> >
> > Hi, all
> >
> > Commit 2633dae2e487 standardized LSN formatting with zero-padding. However,
> > a mistake was made (by me) in the example within a comment.
> >
> > This patch provides a fix for that.
> >
> > diff --git a/src/include/access/xlogdefs.h b/src/include/access/xlogdefs.h
> > index 514f03df0b6..2397fb24115 100644
> > --- a/src/include/access/xlogdefs.h
> > +++ b/src/include/access/xlogdefs.h
> > @@ -38,7 +38,7 @@ typedef uint64 XLogRecPtr;
> > /*
> > * Handy macro for printing XLogRecPtr in conventional format, e.g.,
> > *
> > - * printf("%X/08X", LSN_FORMAT_ARGS(lsn));
> > + * printf("%X/%08X", LSN_FORMAT_ARGS(lsn));
> > *
> > * To avoid breaking translatable messages, we're directly applying the
> > * LSN format instead of using a macro.
>
> LGTM.
> Barring any objections, I'll commit the patch.

Pushed. Thanks!

Regards,

--
Fujii Masao

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Japin Li 2025-08-14 02:21:56 Re: Update the LSN format in the comment example
Previous Message Japin Li 2025-08-14 02:11:07 Re: [WIP]Vertical Clustered Index (columnar store extension) - take2