Andy Balholm wrote:
On May 26, 2010, at 11:18 AM, Kevin Grittner wrote:
>> Do you want to package this up as a patch for 9.1? If not, is it
>> OK if I do?
> I'm not quite sure how to go about changing it from an add-on
> function to a built-in one. So if you want to do that, go ahead. If
> you'd rather I did, just tell me how it's done.
You would basically move the functions and their prototypes to cash.c
and cash.h, and then (instead of CREATE FUNCTION, etc.) add
corresponding entries to pg_proc.h and pg_operator.h. (If I'm
missing something, someone please jump in.) Of course there's the
issue of adding the new operators to the documentation, too.
You would then generate a diff in context format and post to the
-hackers list with that file as an attachment. Don't forget to add
it to the "CommitFest" page:
https://commitfest.postgresql.org/action/commitfest_view/open
If you're going to do this, be sure to breeze through the developer's
FAQ:
http://wiki.postgresql.org/wiki/Developer_FAQ
Of course, if that's all too daunting, I can do it, but it seems to
me that you've already done the hard part (in creating working
functions), so I figured it might be satisfying for you to see it
through to the end.
-Kevin
