Scott Goodwin <[EMAIL PROTECTED]> writes: > If anyone cares to, I'd still like feedback on anything I could have > done better in this patch
Since, you asked, two minor points: (1) palloc() is for memory allocation in the backend; use malloc() (or a wrapper like pg_malloc()) in frontend apps and libpq (2) naming variables like word1_Word2 is inconsistent with the surrounding code. -Neil ---------------------------(end of broadcast)--------------------------- TIP 5: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faqs/FAQ.html