Lists: | pgsql-patches |
---|
From: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> |
---|---|
To: | PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Clarify libpq docs |
Date: | 2004-01-26 05:27:49 |
Message-ID: | Pine.LNX.4.58.0401261623320.4192@linuxworld.com.au |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
The attached patch clarifies (or, rather, makes explicit) to readers how
to handle memory management for char pointers returned by libpq functions.
Although the sections on PQfinish(), PQclear() and PQfreemem() give an
indication that all pointers returned by functions point to memory
allocated in the underlying library, I thought it would be useful to make
this clear.
Gavin
Attachment | Content-Type | Size |
---|---|---|
libpq.sgml.patch | text/plain | 4.8 KB |
From: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> |
---|---|
To: | PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Examples for PREPARE/EXECUTE docs |
Date: | 2004-01-26 07:54:35 |
Message-ID: | Pine.LNX.4.58.0401261854080.5577@linuxworld.com.au |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Attached is a patch adding a few examples for PREPARE/EXECUTE
Gavin
Attachment | Content-Type | Size |
---|---|---|
prepare-docs.patch | text/plain | 2.0 KB |
From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Clarify libpq docs |
Date: | 2004-01-26 16:02:27 |
Message-ID: | 200401261702.28000.peter_e@gmx.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Gavin Sherry wrote:
> The attached patch clarifies (or, rather, makes explicit) to readers
> how to handle memory management for char pointers returned by libpq
> functions. Although the sections on PQfinish(), PQclear() and
> PQfreemem() give an indication that all pointers returned by
> functions point to memory allocated in the underlying library, I
> thought it would be useful to make this clear.
Please don't put tabs in SGML files. They don't have any real meaning
there. Use spaces to indent. (Normally, we indent code by 4 spaces
and SGML by 1 space.)
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Clarify libpq docs |
Date: | 2004-01-26 16:45:49 |
Message-ID: | 8218.1075135549@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> Please don't put tabs in SGML files.
That's a new one on me. Why should we avoid tabs? The existing files
are certainly full of them.
regards, tom lane
From: | Neil Conway <neilc(at)samurai(dot)com> |
---|---|
To: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> |
Cc: | PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Examples for PREPARE/EXECUTE docs |
Date: | 2004-01-26 17:30:47 |
Message-ID: | 873ca2mxbs.fsf@mailbox.samurai.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> writes:
> Attached is a patch adding a few examples for PREPARE/EXECUTE
Patch applied, with a few fixes.
BTW, Bruce and I discussed whether it would be helpful for me to apply
any patches -- and if so, what the proper procedure for doing so
is. What we decided was that I can help out Bruce's patch application
workload (and reduce application latency) by applying a few patches on
occasion. However, I'll only apply relatively small patches that (a)
seem pretty safe and correct (b) I've reviewed myself. Any
comments/objections to this procedure are welcome.
-Neil
From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Clarify libpq docs |
Date: | 2004-01-26 18:02:59 |
Message-ID: | 200401261902.59663.peter_e@gmx.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Tom Lane wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> > Please don't put tabs in SGML files.
>
> That's a new one on me. Why should we avoid tabs? The existing
> files are certainly full of them.
Those are just the files that I haven't edited recently and gave the old
"M-x untabify" treatment...
I ask that tabs be avoided for two reasons: One, we use a one-character
indentation in the source, so it's no use that only every 8th
indentation level is made by a tab, it's just annoying when you're
editing. (The only other indentation scheme for SGML/XML that I see
used is two spaces, so I don't think going to 4 or 8 spaces is in
question.) Two, when rendering, the tabs have questionable meaning.
In HTML they are treated like a single space, who knows what they mean
in a PDF with variable width fonts.
From: | Neil Conway <neilc(at)samurai(dot)com> |
---|---|
To: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> |
Cc: | PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Clarify libpq docs |
Date: | 2004-01-26 22:14:30 |
Message-ID: | 87llnumk6x.fsf@mailbox.samurai.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> writes:
> The attached patch clarifies (or, rather, makes explicit) to readers how
> to handle memory management for char pointers returned by libpq
> functions.
Patch applied. I expanded the tabs in that file to 4 spaces, per the
discussion. I also noticed and fixed a few trivial inconsistencies in
the libpq function prototypes.
Thanks for the patches, Gavin.
-Neil