Re: libpq docs about PQfreemem

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: libpq docs about PQfreemem
Date: 2007-02-04 12:02:38
Message-ID: 45C5CB5E.3040101@hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Peter Eisentraut wrote:
> Magnus Hagander wrote:
>> Now, there are two options for this. Either we fix it (I can put
>> together a patch), or we remove it altogether. To me, it seems to be
>> just an implementation detail and some kind of explanation why we're
>> doing it - which would live better in a source code comment than in
>> the docs.
>
> Old code assumes you can use free() to free all of these things, so it
> seems reasonable to give some background about why that is not the best
> method anymore.

Ok. So you say update it to be worded to cover the correct info about
win32? Or should we change it into something that says "previously you
could use free(), but for portability reasons..."etc?

//Magnus

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jan Wieck 2007-02-04 15:06:27 Re: Proposal: Commit timestamp
Previous Message Peter Eisentraut 2007-02-04 11:46:08 Re: libpq docs about PQfreemem