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
