Re: IntArray in c.h

From: Hitoshi Harada <umi(dot)tanuki(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: IntArray in c.h
Date: 2009-12-30 02:46:40
Message-ID: e08cc0400912291846l56f7ee5bo94ee1650cab51f55@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2009/12/30 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:
> Hitoshi Harada <umi(dot)tanuki(at)gmail(dot)com> writes:
>> I found the struct IntArray defined in c.h is actually used only in
>> execQual.c. ISTM the definition should be at least moved to the right
>> place.
>
> It's a general-purpose datatype that might be used anywhere that array
> indexing happens.  I think the fact that it's currently used only in
> execQual is mere happenstance, and should not be "enforced" by moving
> or removing the declaration.

I would be convinced if the struct or the logic was complex, but
actually it is so simple that it can be replaced by primitive int
array. Also, it seems to me that c.h is too general place to declare
it for such purpose. Does nobody else think so?

Regards,

--
Hitoshi Harada

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2009-12-30 02:48:11 Re: [PATCH 4/4] Add tests to dblink covering use of COPY TO FUNCTION
Previous Message Hiroshi Inoue 2009-12-30 02:29:50 msvc build fails in Japanese environment