Re: Review Report: propose to include 3 new functions into intarray and intagg

Markus Wanner <markus@bluegap.ch> writes:
> Submission review: we generally prefer having patches archived on our
> mailing lists, so please just send future patches or revisions of this
> patch to our lists (I prefer -hackers, but probably -patches is still
> the official one).

Please.  But -patches is dead, please use -hackers.

> The documentation should go into a README file of the
> contrib module or some such.

No, it should go into the SGML docs, specifically doc/src/sgml/intarray.sgml
and intagg.sgml.  We got rid of flat-text README documentation for
contrib in 8.3, and are not about to permit any backsliding.

> Summary: My general feeling is, that this patch should be applied after 
> minor code style corrections. As a longer term goal I think intagg 
> should be integrated into  core, since it's very basic functionality. 

Well, what should be integrated is generic (non-datatype-specific)
versions of this functionality, ie functions working on
anyarray/anyelement not just int4[]/int4.  There was some discussion
about that a few days ago.  I'm not really sure how painful it would be
to do; probably the generalization to non-fixed-width types would be the
trickiest part.
        regards, tom lane