Re: xmalloc => pg_malloc - Mailing list pgsql-hackers

From Tom Lane
Subject Re: xmalloc => pg_malloc
Date
Msg-id 8988.1349195433@sss.pgh.pa.us
Whole thread Raw
In response to Re: xmalloc => pg_malloc  (Andres Freund <andres@2ndquadrant.com>)
Responses Re: xmalloc => pg_malloc  (Phil Sorber <phil@omniti.com>)
Re: xmalloc => pg_malloc  (Andres Freund <andres@2ndquadrant.com>)
List pgsql-hackers
Andres Freund <andres@2ndquadrant.com> writes:
>> pg_calloc    (randomly different API for pg_malloc0)

> Do we need this?

I thought about getting rid of it, but there are some dozens of calls
scattered across several files, so I wasn't sure it was worth it.
Anybody else have an opinion?

> I wonder whether the same set of functions should also be available in the 
> backend with ereport(EC_OUT_OF_MEMORY, ...) behaviour as well.

In the backend, you almost always ought to be using palloc instead.
The only places where it's really appropriate to be using malloc
directly are where you don't want an error thrown for out-of-memory.
So I think providing these in the backend would do little except to
encourage bad programming.
        regards, tom lane



pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: xmalloc => pg_malloc
Next
From: Phil Sorber
Date:
Subject: Re: xmalloc => pg_malloc