Re: [BUG] Storage declaration in ECPG - Mailing list pgsql-hackers

From Kyotaro Horiguchi
Subject Re: [BUG] Storage declaration in ECPG
Date
Msg-id 20220905.171210.1866682352957653815.horikyota.ntt@gmail.com
Whole thread Raw
List pgsql-hackers
At Sun, 04 Sep 2022 13:49:53 +0300, Andrey Sokolov <a.sokolov@arenadata.io> wrote in 
> Hi,
> 
> The ECPG preprocessor converts the code
> "static VARCHAR str1[10], str2[20], str3[30];"
> into
> "static  struct varchar_1  { int len; char arr[ 10 ]; }  str1 ;
>          struct varchar_2  { int len; char arr[ 20 ]; }  str2 ;
>          struct varchar_3  { int len; char arr[ 30 ]; }  str3 ;".
> Storage declaration applies only to the first structure.

Good catch!

> The patch in the attachment fixes the bug. Storage declaration will be
> repeated before each structure.
> The patch is on github too:
> https://github.com/andr-sokolov/postgresql/commit/c8f8fc7a211938569e7d46c91a428d8cb25b6f9c

And the code looks good to me.

About the test, don't we need the test for non-varchar/bytea static
variables like "static int inta, intb, intc;"?

regards.

-- 
Kyotaro Horiguchi
NTT Open Source Software Center



pgsql-hackers by date:

Previous
From: Richard Guo
Date:
Subject: Re: [POC] Allow flattening of subquery with a link to upper query
Next
From: Daniel Gustafsson
Date:
Subject: Re: Fix typo function circle_same (src/backend/utils/adt/geo_ops.c)