Horton

Re: Size and size_t in dsa API

Thomas Munro <thomas.munro@enterprisedb.com>

View thread View on postgr.es
From: Thomas Munro <thomas.munro@enterprisedb.com>
To: "Ideriha, Takeshi" <ideriha.takeshi@jp.fujitsu.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2018-09-21T12:56:47Z
Lists: pgsql-hackers 
On Thu, Sep 20, 2018 at 10:13 PM Ideriha, Takeshi
<ideriha.takeshi@jp.fujitsu.com> wrote:
> >> As a non-expert developer's opinion, I think mixing of Size and size_t makes difficult
> >to understand source code.
> >
> >Agreed.  Let's change them all to size_t and back-patch that to keep future
> >back-patching easy.  Patch attached.
>
> Thank you for the quick action. I'm happy now.
> I confirmed English word Size in comment is kept as it is.

Pushed.  Thanks for the reminder about that.

-- 
Thomas Munro
http://www.enterprisedb.com

Commits

  1. Use size_t consistently in dsa.{ch}.