Bruce Momjian writes:
> I found it was easier to concatenate the "UNION" at the macro call site,
> rather than the macro body. This eliminates the extra macro. Patch
> attached and applied.
Yeah, that looks like a much safer idea --- avoids making assumptions
about the behavior of the macro exp
I found it was easier to concatenate the "UNION" at the macro call site,
rather than the macro body. This eliminates the extra macro. Patch
attached and applied.
---
pgman wrote:
>
> Modified patch attached and applied.
>
Modified patch attached and applied.
Rather than create the lists in psql, I used UNION SELECT 'KEYWORD' to
pass the keyword to the backend to be added to the query list.
This was already being done for schema names, and was easy and efficient
to add. My addition is even simpler because it just
On Fri, Feb 03, 2006 at 04:15:32PM -0500, Tom Lane wrote:
> Joachim Wieland <[EMAIL PROTECTED]> writes:
> > - Should all the
> > const char* foocompletion = { "BAR", "BAZ" };
> > be changed to
> > char* foocompletion = { "BAR", "BAZ" };
> Removing const decoration sure seems like the wro
On Fri, Feb 03, 2006 at 04:15:32PM -0500, Tom Lane wrote:
> > - Should all the
> > const char* foocompletion = { "BAR", "BAZ" };
> > be changed to
> > char* foocompletion = { "BAR", "BAZ" };
> Removing const decoration sure seems like the wrong direction to be
> going in. Why can't you
Joachim Wieland <[EMAIL PROTECTED]> writes:
> - Should all the
> const char* foocompletion = { "BAR", "BAZ" };
> be changed to
> char* foocompletion = { "BAR", "BAZ" };
Removing const decoration sure seems like the wrong direction to be
going in. Why can't you still have const on th
On Fri, Feb 03, 2006 at 01:59:19PM -0500, Bruce Momjian wrote:
> I will adjust this patch based on later feedback emails.
I have another version of the patch. Minor style errors are corrected in it.
As far as I can see, open items are:
- Should all the
const char* foocompletion = { "BAR
I will adjust this patch based on later feedback emails.
Your patch has been added to the PostgreSQL unapplied patches list at:
http://momjian.postgresql.org/cgi-bin/pgpatches
It will be applied as soon as one of the PostgreSQL committers reviews
and approves it.
--
Neil,
thanks for your review. I accepted what you wrote for all items I don't
mention in this reply.
On Sat, Jan 07, 2006 at 10:59:27PM -0500, Neil Conway wrote:
> > + #define COMPLETE_WITH_MALLOCED_LIST(list) \
> > + do { COMPLETE_WITH_LIST((const char**) list); free(list); list
> > = (cha
A few minor stylistic gripes...
On Fri, 2006-01-06 at 20:27 +0100, Joachim Wieland wrote:
> *** 150,155
> --- 151,171
> do {completion_charp = Query_for_list_of_attributes;
> completion_info_charp = table; matches = completion_matches(text,
> complete_from_query); } while(0)
>
> /
10 matches
Mail list logo