| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Size vs size_t |
| Date: | 2017-03-16 21:01:34 |
| Message-ID: | 20170316210134.4si3zeuizmev7j73@alap3.anarazel.de |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2017-03-16 16:59:29 -0400, Robert Haas wrote:
> On Thu, Mar 16, 2017 at 4:40 PM, Thomas Munro
> <thomas(dot)munro(at)enterprisedb(dot)com> wrote:
> > Noticing that the assembled hackers don't seem to agree on $SUBJECT in
> > new patches, I decided to plot counts of lines matching \<Size\> and
> > \<size_t\> over time. After a very long run in the lead, size_t has
> > recently been left in the dust by Size.
>
> I guess I assumed that we wouldn't have defined PG-specific types if
> we wanted to just use the OS-supplied ones.
I think, in this case, defining Size in the first place was a bad call
on behalf of the project. It gains us absolutely nothing, but makes it
harder to read for people that don't know PG all that well. I think we
should slowly phase out Size usage, at least in new code.
Greetings,
Andres Freund
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2017-03-16 21:10:09 | Re: Size vs size_t |
| Previous Message | Stephen Frost | 2017-03-16 21:01:31 | Re: new set of psql patches for loading (saving) data from (to) text, binary files |