From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Danny Milosavljevic <danny(dot)milosavljevic(at)fabalabs(dot)org> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: libpq-fe: PQgetvalue() ? |
Date: | 2006-11-24 23:44:21 |
Message-ID: | 200611242344.kAONiL603572@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
This has been saved for the 8.3 release:
http://momjian.postgresql.org/cgi-bin/pgpatches_hold
---------------------------------------------------------------------------
Danny Milosavljevic wrote:
> Hi,
>
> I'm replying to a thread that is two years old (
> http://archives.postgresql.org/pgsql-sql/2004-10/msg00099.php
> ), but:
>
> >Tom Lane wrote at Fri, 15 Oct 2004 10:21:29 -0400:
> >>Christoph Haller <ch ( at ) rodos ( dot ) fzk ( dot ) de> writes:
> >> You cannot PQclear(pgresult_varible) while cstring_varible is in use.
> >> You do not need to free cstring_variable, PQclear(pgresult_varible)
> will do.
> >> I personally prefer to allocate local memory, "strcpy" PQgetvalue,
> >> and then PQclear. But that's a matter of taste, I suppose.
>
> >That is surely overkill.
>
> >PQgetvalue doesn't allocate new memory for its result. The docs are
> >reasonably clear, I thought:
>
> > The pointer returned by PQgetvalue points to storage that is part
> > of the PGresult structure. One should not modify the data it points
> > to, and one must explicitly copy the data into other storage if it
> > is to be used past the lifetime of the PGresult structure itself.
> >
> > regards, tom lane
>
> Shouldn't the return type of "PQgetvalue" be "const char*", then?
>
> Maybe its just me...
>
> regards,
> Danny Milosavljevic
[ Attachment, skipping... ]
>
> ---------------------------(end of broadcast)---------------------------
> TIP 2: Don't 'kill -9' the postmaster
--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2006-11-24 23:52:03 | Re: libpq-fe: PQgetvalue() ? |
Previous Message | Bruce Momjian | 2006-11-24 23:43:36 | Re: [HACKERS] [PERFORM] Direct I/O issues |