Re: [PATCH] Add tests for Bitmapset

Поиск
Список
Период
Сортировка
От David Rowley
Тема Re: [PATCH] Add tests for Bitmapset
Дата
Msg-id CAApHDvpn3bshX+_2t0NbJ263r1etcBdmkROjuJw8DvyZkMcXqg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] Add tests for Bitmapset  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: [PATCH] Add tests for Bitmapset
Список pgsql-hackers
On Thu, 9 Oct 2025 at 15:13, Michael Paquier <michael@paquier.xyz> wrote:
> What do you think about the attached?

Thanks. Looks pretty good.

> + members = palloc(sizeof(int) * num_ops);

Any reason to pfree that and allocate that to the same size as it already was?

Wondering if the "members[pos] = members[--num_members];" is worth a
short comment. Maybe something like: /* zap this member by moving the
final array member into its place and shrinking the array by 1 */

David



В списке pgsql-hackers по дате отправления: