NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit4bc4cfe

committed

Suppress -Wunused-result warning for strtol().

I'm not sure which bozo thought it's a problem to use strtol() onlyfor its endptr result, but silence the warning using same methodused elsewhere.Report: <f845d3a6-5328-3e2a-924f-f8e91aa2b6d2@2ndquadrant.com>

1 parent7f61fd1 commit4bc4cfeCopy full SHA for 4bc4cfe

File tree

1 file changed

-1

lines changed

src/backend/utils/adt
- dbsize.c

1 file changed

-1

lines changed

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

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -759,13 +759,15 @@ pg_size_bytes(PG_FUNCTION_ARGS)`
`759`	`759`	`/* Part (4): optional exponent */`
`760`	`760`	`if (endptr=='e'\|\|endptr=='E')`
`761`	`761`	`{`
	`762`	`+longexponent;`
`762`	`763`	`char*cp;`
`763`	`764`
`764`	`765`	`/*`
`765`	`766`	`* Note we might one day support EB units, so if what follows 'E'`
`766`	`767`	`* isn't a number, just treat it all as a unit to be parsed.`
`767`	`768`	`*/`
`768`		`-(void)strtol(endptr+1,&cp,10);`
	`769`	`+exponent=strtol(endptr+1,&cp,10);`
	`770`	`+(void)exponent;/* Silence -Wunused-result warnings */`
`769`	`771`	`if (cp>endptr+1)`
`770`	`772`	`endptr=cp;`
`771`	`773`	`}`

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!

Commit4bc4cfe

File tree

1 file changed

1 file changed

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

0 commit comments