minor code question: portal memory cxts

Started by Neil Conwayover 21 years ago3 messageshackers
Jump to latest
#1Neil Conway
neilc@samurai.com

The memory context created at src/backend/utils/mmgr/portalmem.c:183
shares the name of the memory context created at portalmem.c:279 (they
are both called "PortalHeapMemory"). Is there a reason for this?

-Neil

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Neil Conway (#1)
Re: minor code question: portal memory cxts

Neil Conway <neilc@samurai.com> writes:

The memory context created at src/backend/utils/mmgr/portalmem.c:183
shares the name of the memory context created at portalmem.c:279 (they
are both called "PortalHeapMemory"). Is there a reason for this?

Copy-and-paste oversight I'd say. Probably the latter ought to be
"PortalHoldContext" or some such.

regards, tom lane

#3Neil Conway
neilc@samurai.com
In reply to: Tom Lane (#2)
Re: minor code question: portal memory cxts

On Tue, 2004-10-12 at 09:40, Tom Lane wrote:

Copy-and-paste oversight I'd say. Probably the latter ought to be
"PortalHoldContext" or some such.

Thanks, that's what I suspected. I've applied the attached fix to HEAD.

-Neil

Attachments:

portal-mem-cntx-name-1.patchtext/x-patch; charset=iso-8859-1; name=portal-mem-cntx-name-1.patchDownload+2-2