Re: Use allocation macros in the logical replication code

[Peter Smith]

On Wed, Feb 4, 2026 at 1:47 PM Hayato Kuroda (Fujitsu)
<kuroda.hay...@fujitsu.com> wrote:
>
> Dear Peter,
>
> Thanks for the patch. I have one comment for the patch:
> ```
> @@ -836,8 +836,9 @@ SnapBuildAddCommittedTxn(SnapBuild *builder, 
> TransactionId xid)
>                 elog(DEBUG1, "increasing space for committed transactions to 
> %u",
>                          (uint32) builder->committed.xcnt_space);
>
> -               builder->committed.xip = repalloc(builder->committed.xip,
> -                                                                             
>     builder->committed.xcnt_space * sizeof(TransactionId));
> +               builder->committed.xip = 
> repalloc_array(builder->committed.xip,
> +                                                       TransactionId,
> +                                                       
> builder->committed.xcnt_space);
> ```
>
> Number of tabs seems not to enough here. Other than that LGTM.
>

Thanks for the feedback. Now I have run pg_indent on all the patched files.

PSA v2

======
Kind Regards,
Peter Smith.
Fujitsu Australia

v2-0001-use-alloc-macros-in-logical-replication-code.patch
Description: Binary data