Re: Major problem with custom data type

From: "Morgan Kita" <mkita(at)verseon(dot)com>
To: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: <pgsql-novice(at)postgresql(dot)org>
Subject: Re: Major problem with custom data type
Date: 2005-04-04 20:15:37
Message-ID: 08B420FF5BF7BC42A064212C2EB768801C1094@neutron.verseon.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-novice


"Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:

>There's nothing obviously wrong in what you posted ... other than the
>very poor style of misspelling "VARHDRSZ" as "4", not using VARDATA and
>VARATT_SIZEP macros, and generally doing your best to ignore every one
>of the portability conventions that are built into the Postgres
>sources. This code *will* break at some point in the future, but
>it's not immediately obvious why it's not broken on a typical
>present-day 32-bit-Intel platform. (Now if that's not what you are
>running on, maybe you should mention it...)

>When there's nothing broken in what you're looking at, that's usually a
>good sign that the problem is elsewhere. Could you post a complete
>non-working example rather than just extracts?

Hmm I was not aware of those macros, or that I was breaking portability conventions for postgres. If you could direct me as to where I could read up on these things, I would be more than willing to learn about them.

I actually figured my own problem out, thank you for pointing out that is was likely not in the code, I found I was missing one of the declaration statements needed by postgres.

Once again any links to where I can read up on postgres coding conventions would be great as I have to write a few more of these custom data types. I have read through the manual but I am guessing that I missed a section.

Thanks,
Morgan

Responses

Browse pgsql-novice by date

  From Date Subject
Next Message John DeSoi 2005-04-04 23:47:36 Re: I am verynew to postgresql problem with defining blob field
Previous Message John DeSoi 2005-04-04 18:19:47 Re: Disconnect - required, recommended, or optional?