Make ecpg's rjulmdy() and rmdyjul() agree with their declarations.

tglsfdc · tglsfdc · commit edb39e321049 · 2021-01-28T11:17:47.000-05:00
We had "short *mdy" in the extern declarations, but "short mdy[3]" in the actual function definitions. Per C99 these are equivalent, but recent versions of gcc have started to issue warnings about the inconsistency. Clean it up before the warnings get any more widespread. Back-patch, in case anyone wants to build older PG versions with bleeding-edge compilers. Discussion: https://postgr.es/m/2401575.1611764534@sss.pgh.pa.us
diff --git a/src/interfaces/ecpg/compatlib/informix.c b/src/interfaces/ecpg/compatlib/informix.c
@@ -533,7 +533,7 @@ rtoday(date * d)
 }
 
 int
-rjulmdy(date d, short mdy[3])
+rjulmdy(date d, short *mdy)
 {
 	int			mdy_int[3];
 
@@ -584,7 +584,7 @@ rfmtdate(date d, char *fmt, char *str)
 }
 
 int
-rmdyjul(short mdy[3], date * d)
+rmdyjul(short *mdy, date * d)
 {
 	int			mdy_int[3];
 

Original file line number	Diff line number	Diff line change
`@@ -533,7 +533,7 @@ rtoday(date * d)`
`533`	`533`	`}`
`534`	`534`
`535`	`535`	`int`
`536`		`-rjulmdy(date d, short mdy[3])`
	`536`	`+rjulmdy(date d, short *mdy)`
`537`	`537`	`{`
`538`	`538`	`int mdy_int[3];`
`539`	`539`
`@@ -584,7 +584,7 @@ rfmtdate(date d, char fmt, char str)`
`584`	`584`	`}`
`585`	`585`
`586`	`586`	`int`
`587`		`-rmdyjul(short mdy[3], date * d)`
	`587`	`+rmdyjul(short mdy, date d)`
`588`	`588`	`{`
`589`	`589`	`int mdy_int[3];`
`590`	`590`