NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commitb46fa32

Michael Meskes

committed

ECPG: Make the preprocessor emit ';' if the variable type for a list of

variables is varchar. This fixes this test case:int main(void){ exec sql begin declare section; varchar a[50], b[50]; exec sql end declare section; return 0;}Since varchars are internally turned into custom structs andthe type name is emitted for these variable declarations,the preprocessed code previously had:struct varchar_1 { ... } a _,_ struct varchar_2 { ... } b ;The comma in the generated C file was a syntax error.There are no regression test changes since it's not exercised.Patch by Boszormenyi Zoltan <zb@cybertec.at>

1 parent37364c6 commitb46fa32Copy full SHA for b46fa32

File tree

1 file changed

-1

lines changed

src/interfaces/ecpg/preproc
- ecpg.trailer

1 file changed

-1

lines changed

`‎src/interfaces/ecpg/preproc/ecpg.trailer`

Lines changed: 6 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -837,7 +837,12 @@ opt_signed: SQL_SIGNED`
`837`	`837`	`variable_list: variable`
`838`	`838`	`{ $$ = $1; }`
`839`	`839`	`\| variable_list ',' variable`
`840`		`-{ $$ = cat_str(3, $1, mm_strdup(","), $3); }`
	`840`	`+{`
	`841`	`+if (actual_type[struct_level].type_enum == ECPGt_varchar)`
	`842`	`+$$ = cat_str(3, $1, mm_strdup(";"), $3);`
	`843`	`+else`
	`844`	`+$$ = cat_str(3, $1, mm_strdup(","), $3);`
	`845`	`+}`
`841`	`846`	`;`
`842`	`847`
`843`	`848`	`variable: opt_pointer ECPGColLabel opt_array_bounds opt_bit_field opt_initializer`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitb46fa32

File tree

1 file changed

1 file changed

`‎src/interfaces/ecpg/preproc/ecpg.trailer`

0 commit comments