| From: | Gregory Stark <stark(at)enterprisedb(dot)com> |
|---|---|
| To: | "Jeremy Drake" <pgsql(at)jdrake(dot)com> |
| Cc: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Neil Conway" <neilc(at)samurai(dot)com>, "Peter Eisentraut" <peter_e(at)gmx(dot)net>, "David Fetter" <david(at)fetter(dot)org>, "Alvaro Herrera" <alvherre(at)commandprompt(dot)com>, "PostgreSQL Patches" <pgsql-patches(at)postgresql(dot)org>, "Mark Dilger" <pgsql(at)markdilger(dot)com> |
| Subject: | Re: patch adding new regexp functions |
| Date: | 2007-03-21 12:34:12 |
| Message-ID: | 87r6rihj23.fsf@stark.xeocode.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-patches |
"Jeremy Drake" <pgsql(at)jdrake(dot)com> writes:
> BTW, should I be calling get_typlenbyvalalign on TEXTOID or are there macros
> for those also?
Hardcoding -1 for typlen of varlenas is one of the few (the only?) magic
constants used throughout the source code. I'm surprised there isn't a macro
for it though.
Do you need the alignment? If so I want to check the code against the packed
varlena patch. Just in case.
--
Gregory Stark
EnterpriseDB http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Gregory Stark | 2007-03-21 12:51:38 | Re: patch adding new regexp functions |
| Previous Message | Martijn van Oosterhout | 2007-03-21 12:26:55 | Re: Question about the TODO, numerics, and division |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Gregory Stark | 2007-03-21 12:51:38 | Re: patch adding new regexp functions |
| Previous Message | Heikki Linnakangas | 2007-03-21 10:18:07 | Re: Bitmapscan changes - Requesting further feedback |