Improper use about DatumGetInt32

From: "Hou, Zhijie" <houzj(dot)fnst(at)cn(dot)fujitsu(dot)com>
To: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Improper use about DatumGetInt32
Date: 2020-09-21 01:17:21
Message-ID: d8f6bdd536df403b9b33816e9f7e0b9d@G08CNEXMBPEKD05.g08.fujitsu.local
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi

In (/contrib/bloom/blutils.c:277), I found it use DatumGetInt32 to get UInt32 type.
Is it more appropriate to use DatumGetUInt32 here?

See the attachment for the patch

Bes regards,
houzj

Attachment Content-Type Size
0001-DatumGetUInt32.patch application/octet-stream 915 bytes

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Munro 2020-09-21 02:19:33 Re: Handing off SLRU fsyncs to the checkpointer
Previous Message Tomas Vondra 2020-09-21 00:10:30 Re: Planner, check if can use consider HASH for groupings (src/backend/optimizer/plan/planner.c)