NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit7cac630

committed

Fix compiler warning in mul_var_short().

Some compilers (e.g., gcc before version 7) mistakenly think "carry"might be used uninitialized.Reported by Tom Lane, per various buildfarm members, e.g. arowana.

1 parent8daa62a commit7cac630Copy full SHA for 7cac630

File tree

1 file changed

-2

lines changed

src/backend/utils/adt
- numeric.c

1 file changed

-2

lines changed

`‎src/backend/utils/adt/numeric.c‎`

Lines changed: 1 addition & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -8969,7 +8969,7 @@ mul_var_short(const NumericVar var1, const NumericVar var2,`
`8969`	`8969`	`intres_ndigits;`
`8970`	`8970`	`NumericDigit*res_buf;`
`8971`	`8971`	`NumericDigit*res_digits;`
`8972`		`-uint32carry;`
	`8972`	`+uint32carry=0;`
`8973`	`8973`	`uint32term;`
`8974`	`8974`
`8975`	`8975`	`/* Check preconditions */`
`@@ -9018,7 +9018,6 @@ mul_var_short(const NumericVar var1, const NumericVar var2,`
`9018`	`9018`	`*res_ndigits = var2ndigits + 1`
`9019`	`9019`	`* ----------`
`9020`	`9020`	`*/`
`9021`		`-carry=0;`
`9022`	`9021`	`for (inti=var2ndigits-1;i >=0;i--)`
`9023`	`9022`	`{`
`9024`	`9023`	`term=PRODSUM1(var1digits,0,var2digits,i)+carry;`

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!

Commit7cac630

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/numeric.c‎`

0 commit comments